Add core_version_requirement: ^8 || ^9 to info.yml to designate that the module is compatible with Drupal 9.

Comments

ankush_03’s picture

Status: Active » Needs review
Issue tags: +DIACWApril2020
StatusFileSize
new1.81 KB
piyuesh23’s picture

Status: Needs review » Reviewed & tested by the community

Patch looks good to me.

cchiste’s picture

StatusFileSize
new1.9 KB

I know this is for the 8.x branch but as there isn't a 9.x... #2 is failing to apply on Drupal 9.0.6 with generic Failed to apply error.
The issue seems to be in message_subscribe_example, all other modules install clean.

cchiste’s picture

cchiste’s picture

StatusFileSize
new1.73 KB

Updated patch to include message_subscribe_test, also removed deprecated 'core: 8.x'.

cchiste’s picture

StatusFileSize
new2.37 KB

Locally it says could not apply however the changes do apply (?) Same goes for using this patch link. Removed 'core: 8.x' as deprecated https://www.drupal.org/node/3070687

lakshmi chidambaram’s picture

StatusFileSize
new2.37 KB

#8 Added a patch to remove core from .info and include core_version_requirement

cchiste’s picture

StatusFileSize
new1.61 KB

Final submission, I swear. Apologies for the spam. This patch updates every module and sub-module (- examples) to remove the 'core:' constraint requirement and add the 'core_version_requirement'. Previous patches fail because of the additional `_examples` module that exists in the repo but does not get carried over when added to a project. Tested as applying cleanly to a local D 9.0.6 instance.

  • jhedstrom committed aa6154e on 8.x-1.x authored by CChiste
    Issue #3128876 by CChiste, ankushgautam76@gmail.com, lakshmi chidambaram...
jhedstrom’s picture

Status: Reviewed & tested by the community » Fixed

Thanks!

Status: Fixed » Closed (fixed)

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