Updating the info file to include the core_version_requirement.

Comments

saschaeggi created an issue. See original summary.

saschaeggi’s picture

StatusFileSize
new418 bytes

Adding the patch

saschaeggi’s picture

Status: Active » Reviewed & tested by the community
StatusFileSize
new24.54 KB

Tested, passed.

Set this RTBC :)

saschaeggi’s picture

Issue tags: +Drupal 9 compatibility
geek-merlin’s picture

Status: Reviewed & tested by the community » Postponed (maintainer needs more info)

> Tested, passed.

Which tool did you use? drupal-check?

Does it make sense to include the drupalci.yml mentioned here? https://github.com/mglaman/drupal-check/wiki/Drupal-9-Readiness

saschaeggi’s picture

Status: Postponed (maintainer needs more info) » Needs review

@geek-merlin
I've checked it with upgrade-status: https://www.drupal.org/project/upgrade_status
I don't think it's needed, but I'd rather open a new ticket to explore it if it's necessary for this module.

Cheers

geek-merlin’s picture

Status: Needs review » Reviewed & tested by the community

Ah thanks, i was not aware of this yet. Not that i expected something else as the module is trivial PHP-wise and all the flesh is in JS.

So putting this on the pipeline to commit. Thanks a bunch!

geek-merlin’s picture

Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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