Skip to content

Feature/laravel 13 upgrade#377

Open
Reyez-nl wants to merge 3 commits into
CodeDredd:masterfrom
Reyez-nl:feature/laravel-13-upgrade
Open

Feature/laravel 13 upgrade#377
Reyez-nl wants to merge 3 commits into
CodeDredd:masterfrom
Reyez-nl:feature/laravel-13-upgrade

Conversation

@Reyez-nl

Copy link
Copy Markdown

Issue: Laravel 13 Support

What I did

  • Updated Illuminate packages
  • Updated phpro/soap-client due to removal of deprecated package azjezz/psl.
  • Added dependencies due to the update (symfony/cache, php-soap/ext-soap-engine).
  • Dropped support for 8.1 due to symfony/cache (7x support from 8.2, 8x supports from 8.4)

How to test

  • ran tests
  • used the soap services

JeroenReyez and others added 3 commits March 24, 2026 15:02
…/psl. Added dependencies due to the update (symfony/cache, php-soap/ext-soap-engine). Dropped support for 8.1 due to symfony/cache (7x support from 8.2, 8x supports from 8.4)
@thybag

thybag commented May 1, 2026

Copy link
Copy Markdown

This version LGTM.

@maartenderie

Copy link
Copy Markdown

Yes please!

If I may, I would delete the composer.lock because this is a package. And add composer update to any workflows that need it.

e.g. guzzleHttp; https://github.com/guzzle/guzzle/blob/5af96f374e0ab4ebd747b8310888c99d3adb0a8c/.github/workflows/ci.yml#L40

@Reyez-nl

Copy link
Copy Markdown
Author

Yes please!

If I may, I would delete the composer.lock because this is a package. And add composer update to any workflows that need it.

e.g. guzzleHttp; https://github.com/guzzle/guzzle/blob/5af96f374e0ab4ebd747b8310888c99d3adb0a8c/.github/workflows/ci.yml#L40

As that may be true you still want a standalone package e.g. testing purposes.

As for this package I hope that it will be merged since its already been a few months now.

@MarkLanser

Copy link
Copy Markdown

@CodeDredd Could you review this please? Would be great to have 🙏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants