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

  1. Change the migrate_spreadsheet.info.yml file to remove the PHP version and add the new core requirement key.
  2. Tag a new 2.0.0 major release.
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

DuaelFr created an issue. See original summary.

DuaelFr’s picture

Status: Active » Needs review
FileSize
442 bytes
DuaelFr’s picture

Issue tags: +Drupal 9 compatibility
DuaelFr’s picture

Hi @claudiu.cristea
How can I help move this forward?
It's blocking us to move to D9 for now.
Thanks!

claudiu.cristea’s picture

FileSize
453 bytes
440 bytes

Small improvement.

  • claudiu.cristea committed c826ca6 on 2.0.x
    Issue #3165630 by claudiu.cristea, DuaelFr: Drupal 9 compatibility
    
claudiu.cristea’s picture

Status: Needs review » Fixed

Fixed, thank you!

I've also released 2.0.0

DuaelFr’s picture

Thanks a lot!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.