Skip to content

Conversation

jrfnl
Copy link
Member

@jrfnl jrfnl commented Aug 10, 2025

Release checklist

General

  • Verify, and if necessary, update the version constraints for dependencies in the composer.json - PR #xxx
  • Verify that any new functions have type declarations whenever possible.
  • Add changelog for the release - PR Changelog for PHPCSUtils 1.1.1 #696
    ✏️ Remember to add a release link at the bottom!

Release

  • Merge this PR
  • Make sure all CI builds are green.
  • Tag and create a release (careful, GH defaults to develop!) & copy & paste the changelog to it.
    ✏️ Don't forget to copy the link collection from the bottom of the changelog!
  • Make sure all CI builds are green.
  • Verify that the website regenerated correctly.
  • Close the milestone
  • Open a new milestone for the next release
  • If any open PRs/issues which were milestoned for this release did not make it into the release, update their milestone.
  • Fast-forward develop to be equal to stable

Publicize

  • Tweet about the release.
  • Inform the primary dependants of this repo (PHPCSExtra, WordPressCS, PHPCompatibility and VariableAnalysis) about the release.

github-actions bot and others added 17 commits June 13, 2025 04:01
…date-phpcs-version

GetVersionTest: update for release of PHPCS 3.13.1
…date-phpcs-version

GetVersionTest: update for release of PHPCS 3.13.2
…y declaration

... to document the behaviour.
…s-analyzeoostructure-add-test-with-multi-prop

ObjectDeclarations::analyzeOOStructure(): add test with multi-property declaration
Should work just the same and just as well.

Historical context: looks like the use of `DummyFile` was inspired by following in PHPCS' own footsteps, but was not needed there either...
…case-use-localfile

UtilityMethodTestCase: use LocalFile instead of DummyFile
Remove the condition containing a hard-coded repository name in favour of a more generic condition which should safeguard that the cron job doesn't run on forks just the same.
…-dont-run-cronjobs-on-forks-condition

GH Actions: improve "don't run on forks" condition
…ccessible()

Since PHP 8.1, calling the `Reflection*::setAccessible()` methods is no longer necessary as reflected properties/methods/etc will always be accessible.
However, the method calls are still needed for PHP < 8.1.

As of PHP 8.5, calling the `Reflection*::setAccessible()` methods is now formally deprecated and will yield a deprecation notice, which will fail test runs.
As of PHP 9.0, the `setAccessible()` method(s) will be removed.

With the latter in mind, this commit prevents the deprecation notice by making the calls to `setAccessible()` conditional.

Silencing the deprecation would mean, this would need to be "fixed" again come PHP 9.0, while the current solution should be stable, including for PHP 9.0.

Ref: https://wiki.php.net/rfc/deprecations_php_8_5#extreflection_deprecations
…sible()

Since PHP 8.1, calling the `Reflection*::setAccessible()` methods is no longer necessary as reflected properties/methods/etc will always be accessible.
However, the method calls are still needed for PHP < 8.1.

As of PHP 8.5, calling the `Reflection*::setAccessible()` methods is now formally deprecated and will yield a deprecation notice, which will fail test runs.
As of PHP 9.0, the `setAccessible()` method(s) will be removed.

With the latter in mind, this commit prevents the deprecation notice by making the calls to `setAccessible()` conditional.

Silencing the deprecation would mean, this would need to be "fixed" again come PHP 9.0, while the current solution should be stable, including for PHP 9.0.

Ref: https://wiki.php.net/rfc/deprecations_php_8_5#extreflection_deprecations
…deprecations-php-8.5

PHP 8.5 | TestUtils: prevent deprecation notice for Reflection*::setAccessible()
Add `display_startup_errors=On` as per the current recommendation from PHPUnit.

Ref: sebastianbergmann/phpunit-documentation-english@b3b159c
…artup-errors

GH Actions: update PHP ini configuration
@jrfnl jrfnl added this to the 1.1.1 milestone Aug 10, 2025
@jrfnl jrfnl merged commit f7eb16f into stable Aug 10, 2025
68 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

1 participant