Seems like the automated checker doesn't complain about anything but the .info file modification. Patch coming soon.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

marcoscano created an issue. See original summary.

marcoscano’s picture

Assigned: marcoscano » Unassigned
Status: Active » Needs review
FileSize
373 bytes

I haven't manually tested this, just trying to get the first pass with the automated checker green with all modules I have on my site. It seems like with this patch the checker is green as of today.

marcoscano’s picture

A hopefully more complete patch.

Status: Needs review » Needs work

The last submitted patch, 3: 3138582-3.patch, failed testing. View results

KarenS’s picture

Status: Needs work » Needs review
FileSize
10.03 KB

Rerolled

Berdir’s picture

+++ b/schema_article/schema_article.info.yml
@@ -1,7 +1,7 @@
 type: module
 description: Adds Schema.org/Article to the JSON LD array. Creates Article, BlogPosting, SocialMediaPosting, Report, ScholarlyArticle, TechArticle or APIReference.
-core: 8.x
+core_version_requirement: ^8 || ^9
 package: SEO
 dependencies:

the core: key can be kept if you don't require ^8.7.7 or higher. otherwise as you will get an exception on anything lower than 8.7.7 if you remove it.

Also, there was an existing issue: [#310888]

  • KarenS committed d3ea2ab on 8.x-1.x
    Issue #3138582 by mbovan, Berdir, marcoscano, KarenS: Drupal 9...
KarenS’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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