Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Problem/Motivation
Drupal 9 is here (9.0.3 already) and this module has about nothign to do to be compatible according to Drupal Rector.
$ vendor/bin/rector process web/modules/contrib/migrate_spreadsheet --dry-run
Rector v0.7.29
Config file: rector.yml
12/12 [▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓] 100%
[OK] Rector is done!
The thing is that we cannot easily fake composer to install this module on D9 so we'd need it to be marked as compatible.
Being compatible with D9 bumps the php requirement to 7+ so we need to tag a new major version to prevent old websites to upgrade by mistake.
Proposed resolution
- Change the migrate_spreadsheet.info.yml file to remove the PHP version and add the new core requirement key.
- Tag a new 2.0.0 major release.
Comment | File | Size | Author |
---|---|---|---|
#5 | 3165630-5.patch | 440 bytes | claudiu.cristea |
Comments
Comment #2
DuaelFrComment #3
DuaelFrComment #4
DuaelFrHi @claudiu.cristea
How can I help move this forward?
It's blocking us to move to D9 for now.
Thanks!
Comment #5
claudiu.cristeaSmall improvement.
Comment #7
claudiu.cristeaFixed, thank you!
I've also released 2.0.0
Comment #8
DuaelFrThanks a lot!