- Issue created by @project update bot
This is an automated patch generated using Upgrade Status and Drupal Rector. Please see the issue summary for more details. A merge request is also openend and updated.
It is important that any automated tests available are run and that you manually test the changes.
Drupal 11 Compatibility
According to the Upgrade Status module โ these changes make this module compatible with Drupal 11! ๐
Therefore these changes update theinfo.yml
file for Drupal 11 compatibility.Leaving this issue open, even after committing the current patch, will allow the Project Update Bot โ to post additional Drupal 11 compatibility fixes as they become available in Drupal Rector.
Debug info
Bot run #11-121090This patch was created using these packages:
- drupal/upgrade_status: 4.1.0
- mglaman/phpstan-drupal: 1.2.7
- palantirnet/drupal-rector: 0.20.1
- Status changed to Needs work
10 months ago 8:00am 19 March 2024 - ๐ฌ๐งUnited Kingdom james.williams
Might need to add a comment or something to instruct phpstan or whatever it is, to not add that
parent::setUp()
call. - Status changed to Needs review
9 months ago 9:50am 22 April 2024 This is an automated patch generated using Upgrade Status and Drupal Rector. Please see the issue summary for more details. A merge request (MR) is also openend and updated.
It is important that any automated tests available are run and that you manually test the changes.
Drupal 11 Compatibility
According to the Upgrade Status module โ these changes make this module compatible with Drupal 11! ๐
Therefore these changes update theinfo.yml
file for Drupal 11 compatibility.Leaving this issue open, even after committing the current patch or merging the MR, will allow the Project Update Bot โ to post additional Drupal 11 compatibility fixes as they become available in Drupal Rector.
Debug information
Bot run #11-151879These packages were used to generate the fixes:
- drupal/upgrade_status: 4.2.0
- mglaman/phpstan-drupal: 1.2.10
- palantirnet/drupal-rector: 0.20.1
- First commit to issue fork.
MR !10 checked in Drupal 10 and 11, it is working fine. Module is compatible with both Drupal 10 and 11.
- ๐ฌ๐งUnited Kingdom james.williams
I've left some feedback on the review. Generally, why was MR !10 necessary, with changes beyond what was in MR !9?
- ๐ฌ๐งUnited Kingdom james.williams
I have committed the changes from MR !9, alongside a return type compatibility fix that hadn't been flagged up by any of the proposed changes, but was necessary for tests to pass! Shows how 'real' testing is so much better than automated scanning ๐คจ Hopefully that will stop unnecessary changes being proposed now anyway.
- ๐ฌ๐งUnited Kingdom james.williams
I have merged some of the proposed change, but not all of it. Some of the changes are really about compatibility with certain versions of PHP, Symfony, etc, which don't have to be conflated with Drupal 11 compatibility more generally. Thanks for the changes that are useful.
This is an automated patch generated using Upgrade Status and Drupal Rector. Please see the issue summary for more details. A merge request (MR) is also openend and updated.
It is important that any automated tests available are run and that you manually test the changes.
Drupal 11 Compatibility
According to the Upgrade Status module โ , even with these changes, this module is not yet compatible with Drupal 11.
Currently Drupal Rector, version 0.20.3, cannot fix all Drupal 11 compatibility problems.
Therefore, these changes did not update the
info.yml
file for Drupal 11 compatibility.The compatibility issues that Upgrade Status found after the Drupal Rector fixes were applied are attached to help you resolve them manually.
Leaving this issue open, even after committing the current patch or merging the MR, will allow the Project Update Bot โ to post additional Drupal 11 compatibility fixes as they become available in Drupal Rector.
Debug information
Bot run #11-392907These packages were used to generate the fixes:
- drupal/upgrade_status: 4.3.6
- mglaman/phpstan-drupal: 1.3.2
- palantirnet/drupal-rector: 0.20.3
- ๐ฌ๐งUnited Kingdom james.williams
Reverting status, since that last change is explicitly not wanted.