Adding both core and core_version_requirement with anything other than core_version_requirement: ^8 || ^9 will result in an exception.

For example, a module/theme that is compatible with Drupal 8 versions after Drupal 8.8.0 and also Drupal 9 will need to use the following line in its .info.yml file, without a line containing core.

core_version_requirement: ^8.8 || ^9

In the rare occasions the module/theme is also compatible with Drupal 10, its .info.yml file should use the following line.

core_version_requirement: ^8.8 || ^9 || ^10
Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

Harshita mehna created an issue. See original summary.

harshita mehna’s picture

Status: Active » Needs review
StatusFileSize
new387 bytes

I have provided a patch to fix this issue.
Please review it.

Thanks!

priya_degwekar’s picture

StatusFileSize
new53.1 KB
new13.49 KB
new53.71 KB

Hi @Harshita mehna, This patch looks fine.

Adding screenshots for reference.
Thanks.

priya_degwekar’s picture

Status: Needs review » Reviewed & tested by the community

Can be moved to RTBC+1.

krzysztof domański’s picture

Status: Reviewed & tested by the community » Needs work
Issue tags: +Needs reroll

Needs reroll.

mrinalini9’s picture

Status: Needs work » Needs review
Issue tags: -Needs reroll

Patch #2 is being applied cleanly on 3.x branch:

Checking patch fontawesome_iconpicker.info.yml...
Applied patch fontawesome_iconpicker.info.yml cleanly.

So, moving this to Needs review.

Thanks!

avpaderno’s picture

Title: [Drupal 10] Do not allow core: 8.x in info.yml file. » Remove "core: 8.x" from the .info.yml file
Issue summary: View changes
avpaderno’s picture

Category: Bug report » Task
Priority: Normal » Minor
Status: Needs review » Needs work
-core: 8.x
 core_version_requirement: ^8 || ^9 || ^10

^8 must be replaced with ^8.8.

chaitanyadessai’s picture

Status: Needs work » Needs review
StatusFileSize
new447 bytes

Added patch please review.

harshita mehna’s picture

@apaderno, I have created MR !6 please review it.

avpaderno’s picture

Status: Needs review » Reviewed & tested by the community

The MR correctly changes the .info.yml file.

avpaderno’s picture

Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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