Problem/Motivation
Not all dependencies may be PHP 8.2 compatible.
Steps to reproduce
Proposed resolution
Check dependencies for PHP 8.2 compatibility. Update them as needed.
Remaining tasks
- https://github.com/sirbrillig/phpcs-variable-analysis/releases >=2.11.4 #3295520: Update dependencies for Drupal 10.0 and 9.5
- https://github.com/composer/composer/releases/tag/2.3.10 #3298396: Upgrade composer requirement to 2.3.6 to prevent warnings on PHP 8.2
- https://github.com/bovigo/vfsStream/releases/tag/v1.6.11-alpha.0 #3299213: Require mikey179/vfsstream ^1.6.11 to allow tests running on PHP 8.2
- https://github.com/phpstan/phpstan-src/tags >= 1.8.2 #3299606: Update mglaman/phpstan-drupal to 1.1.25 to unblock testing on PHP 8.2
- https://github.com/egulias/EmailValidator/tree/3.2.1 #3298343: upgrade egulias/email-validator to 3.2.1 to support PHP 8.2
User interface changes
no
API changes
Data model changes
no
Comments
Comment #2
gábor hojtsyComment #3
andypostFirst one detected while running tests
mikey179/vfsstreamshould b updated at least to 1.6.11 (unreleased yet) https://github.com/bovigo/vfsStream/commit/88fbc8a0b163fd00375cba0b27d8c...The fix is https://github.com/bovigo/vfsStream/pull/270
Comment #4
andypostOne more issue in sniffers https://github.com/squizlabs/PHP_CodeSniffer/issues/3489
Comment #5
andypostComment #6
andypostFiled PR for PHPStan https://github.com/phpstan/phpstan-src/pull/1478 it's a blocker for #3275851: [META] Fix PHP 8.2 dynamic property deprecations
Comment #7
andypostComment #8
andypostThere's release of
mikey179/vfsstreamto test fix for #3 https://github.com/bovigo/vfsStream/releases/tag/v1.6.11-alpha.0Comment #9
andypostneeds version 2.11.4 with https://github.com/sirbrillig/phpcs-variable-analysis/pull/248
Also phpstan requires 1.8.1 to work with
#[\AllowDynamicProperties]Comment #10
andypostComment #11
andypostto work with
#[AllowDynamicProperties]on PHP 8.2 phpstan will need another upgrade once https://github.com/JetBrains/phpstorm-stubs/pull/1404 landedComment #12
andypostComment #13
andypostFiled #3298343: upgrade egulias/email-validator to 3.2.1 to support PHP 8.2 which needs review to bump core composer's dependency to v3
Comment #14
andypostComposer also needs upgrade, filed #3298396: Upgrade composer requirement to 2.3.6 to prevent warnings on PHP 8.2
Comment #15
andypostAsked for new release https://github.com/sirbrillig/phpcs-variable-analysis/issues/214#issueco...
Comment #16
andyposthttps://github.com/sirbrillig/phpcs-variable-analysis/releases/tag/v2.11.4 release arrived
Comment #17
andypostadded https://github.com/egulias/EmailValidator/tree/3.2.1 to summary
Comment #18
andypostand composer
Comment #19
andypostAlso https://github.com/mglaman/phpstan-drupal/commit/8acaa1e4a15b266cfbd404c... is required to allow phpStan pass
Comment #20
andypostasked to create new stable release https://github.com/bovigo/vfsStream/commit/17d16a85e6c26ce1f3e2fa9ceeacd...
Without a new stable release 10.0.x branch will fail on composer stability test and we can't relax it as beta comming soon https://www.drupal.org/project/drupal/issues/3295821#comment-14620425
but without the fix our CI tests hangs and killed by timeout
Comment #21
catchI think it would be OK to relax stability right up until the beta, it's still a few weeks away. We can have critical follow-ups to move to a stable release.
Comment #22
spokjeComment #23
spokjeComment #24
andypostupdated IS with issues per package, gonna file new one for vfs
Comment #25
andypostComment #26
andypostFiled #3299213: Require mikey179/vfsstream ^1.6.11 to allow tests running on PHP 8.2
Comment #27
spokjeComment #28
spokjeComment #29
spokjeComment #30
andypostTests now running so the only hard blocker is #3299213: Require mikey179/vfsstream ^1.6.11 to allow tests running on PHP 8.2
Comment #31
andypostLast 2 are remaining + https://github.com/sirbrillig/phpcs-variable-analysis/releases/tag/v2.11.4
Comment #32
andypostComment #33
andypostAll requirements merged, so only one upgrade left #3295520: Update dependencies for Drupal 10.0 and 9.5
Comment #34
andypostfix link to issue
Comment #35
andypostOnly general update left ATM to fix sniffers #3295520: Update dependencies for Drupal 10.0 and 9.5
Comment #36
andypostI think it fixed as all 9.5 and 10.0 dependencies are compatibe
Comment #38
andypost