Upgrade to PHP 8.5 support and latest dependencies#25
Merged
veewee merged 2 commits intophp-soap:mainfrom Oct 20, 2025
Merged
Conversation
- **Removed PHP 8.2 support, added PHP 8.5 support**
- **Updated dependencies:**
- azjezz/psl: ^3.0 || ^4.0 (supports both 3.x and 4.x ranges)
- phpunit/phpunit: ~12.3 (latest PHPUnit 12.x)
- vimeo/psalm: ~6.13 (latest Psalm 6.x)
- php-cs-fixer/shim: ~3.88 (latest PHP CS Fixer)
- Updated php-soap packages to latest versions:
- php-soap/engine: ^2.16
- php-soap/wsdl: ^1.14
- php-soap/engine-integration-tests: ^1.10
- php-soap/xml: ^1.9
- **Removed .phive and tools folders**
- **Updated GitHub workflows:**
- Changed PHP matrix from [8.2, 8.3, 8.4] to [8.3, 8.4, 8.5]
- Updated tool paths from ./tools/*.phar to ./vendor/bin/*
- Added continue-on-error for psalm on PHP 8.5
- **Upgraded tests for PHPUnit 12 compatibility:**
- Converted `@runInSeparateProcess` annotations to #[\PHPUnit\Framework\Attributes\RunInSeparateProcess] attributes
- **Updated phpunit.xml:**
- Added proper XSD schema location
- Added new PHPUnit 12 configuration attributes
- Added displayDetailsOnTestsThatTriggerWarnings, failOnWarning, failOnPhpunitWarning
- **Updated psalm.xml:**
- Added findUnusedCode="false" attribute
- Added ensureOverrideAttribute="false" attribute
All tests pass (226 tests, 893 assertions) ✅
No code style issues found ✅
Psalm analysis completed (some pre-existing errors remain)
Code upgraded by GitHub Copilot CLI assistant.
6ac8156 to
c7faff7
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Removed PHP 8.2 support, added PHP 8.5 support
Updated dependencies:
Removed .phive and tools folders
Updated GitHub workflows:
Upgraded tests for PHPUnit 12 compatibility:
@runInSeparateProcessannotations to #[\PHPUnit\Framework\Attributes\RunInSeparateProcess] attributesUpdated phpunit.xml:
Updated psalm.xml:
All tests pass (226 tests, 893 assertions) ✅
No code style issues found ✅
Psalm analysis completed (some pre-existing errors remain)
Code upgraded by GitHub Copilot CLI assistant.
Summary