Skip to content

Conversation

@murrant
Copy link
Owner

@murrant murrant commented Sep 3, 2025

This pull request contains changes for upgrading to PHPUnit 12 automated by the PHPUnit 12 Shift.

Before merging, you need to:

  • Checkout the shift-157632 branch
  • Review all pull request comments for additional changes
  • Run composer update (if the scripts fail, try with --no-scripts)
  • Run your tests suite: vendor/bin/phpunit

If there were changes you feel could have been automated, please send your feedback to [email protected] to help improve Shift for everyone.

@murrant
Copy link
Owner Author

murrant commented Sep 3, 2025

⚠️ PHPUnit 12 requires PHP version 8.3 or higher. Be sure to verify you met this requirement by running php -v in your test environment.

@murrant
Copy link
Owner Author

murrant commented Sep 3, 2025

ℹ️ PHPUnit has documented using return types for test cases and data providers since PHPUnit 8. In an effort to modernize your test suite, Shift added a return type of void to your test cases and a return type of array to your data providers.

Shift understands developers have different preferences when it comes to type hints. If you do not wish to add return types, you may undo this change by running: git revert 5b593f1d

@murrant
Copy link
Owner Author

murrant commented Sep 3, 2025

ℹ️ PHPUnit has documented declaring test classes as final since PHPUnit 8. In an effort to modernize your test suite, Shift has declared your test classes as final.

Shift understands developers have different preferences when it comes to using final. If you do not wish to declare your test classes as final, you may undo this change by running: git revert 71d58daa

@murrant
Copy link
Owner Author

murrant commented Sep 3, 2025

⚗️ This Shift is still being refined. Please report any issues or suggestions to [email protected]. Your feedback is what helps improve the experience for everyone.

@murrant murrant closed this Sep 11, 2025
@murrant murrant deleted the shift-157632 branch October 16, 2025 06:50
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.

3 participants