- First commit to issue fork.
- @grevil opened merge request.
- 🇩🇪Germany Grevil
Fixed using PHPStan and the "upgrade_status" module.
Tested installing the module on Drupal 10 after the changes.
- Status changed to Needs review
about 2 years ago 9:09am 24 January 2023 - 🇩🇪Germany Grevil
Interesting, upgrade_status forgot to mention the test module! Fixed, please review.
- Status changed to Needs work
about 2 years ago 4:27pm 24 January 2023 - 🇩🇪Germany Anybody Porta Westfalica
Thanks @Grevil!
I think all
.info.yml
files should have an equalcore_version_requirement
.
If that's not possible, we should perhaps increase the values for all? - Status changed to Needs review
about 2 years ago 8:07am 25 January 2023 - 🇩🇪Germany Grevil
I think it is fine in this case, as the only info file that differentiates, is for the test module.
- Issue was unassigned.
- Status changed to RTBC
about 2 years ago 3:23pm 30 January 2023 - 🇩🇪Germany Anybody Porta Westfalica
Let's remove the core_version_requirement as of #3096609: Allow contrib test modules to not need a core or core_version_requirement key →
- Status changed to Fixed
about 2 years ago 3:24pm 30 January 2023 Automatically closed - issue fixed for 2 weeks with no activity.