Problem/Motivation

This module has no Drupal 10 compatible release yet. We should fix that.

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Issue fork choices-3335397

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:

About issue forks

Comments

Anybody created an issue. See original summary.

Grevil made their first commit to this issue’s fork.

Grevil opened merge request !18

grevil’s picture

grevil commented

Fixed using PHPStan and the "upgrade_status" module.

Tested installing the module on Drupal 10 after the changes.

grevil’s picture

grevil commented
Status: Active » Needs review

Interesting, upgrade_status forgot to mention the test module! Fixed, please review.

anybody’s picture

anybody
Primary language German
Location Porta Westfalica
commented
Status: Needs review » Needs work

Thanks @Grevil!

I think all .info.yml files should have an equal core_version_requirement.
If that's not possible, we should perhaps increase the values for all?

grevil’s picture

grevil commented
Status: Needs work » Needs review

I think it is fine in this case, as the only info file that differentiates, is for the test module.

anybody’s picture

anybody
Primary language German
Location Porta Westfalica
commented
Assigned: grevil » Unassigned
Status: Needs review » Reviewed & tested by the community

Let's remove the core_version_requirement as of #3096609: Allow contrib test modules to not need a core or core_version_requirement key

  • Anybody committed bc258574 on 2.x authored by Grevil 
    Issue #3335397 by Grevil: Drupal 10 compatibility fixes
anybody’s picture

anybody
Primary language German
Location Porta Westfalica
commented
Status: Reviewed & tested by the community » Fixed

Merged!

Status: Fixed » Closed (fixed)

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