Problem/Motivation

Make the 2.0.x branch compatible with Drupal 10.

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

None

API changes

None

Data model changes

None

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

benjifisher created an issue. See original summary.

benjifisher’s picture

Status: Active » Needs review

I installed the Upgrade Status module. According to its report, this module just needs an update to the .info.yml file.

I ran phpstan locally. It did not find any problems.

I ran the PHPUnit tests, and they showed deprecation errors related to the change record $modules is now a protected property in tests.

I opened a merge request to fix the problems I found.

I also checked coding standards. There are several arrays that extend beyond Column 80. CodeSniffer does not find any other problems. If you would like to fix those as part of this issue, I can update the MR.

bircher’s picture

Thanks!
Yes I will run tests on D10.x and then release a stable version.

Don't worry about the long arrays, they are excluded from phpcs:
https://git.drupalcode.org/project/config_split/-/blob/2.0.x/phpcs.xml.d...
I find it more readable the way it is now.

benjifisher’s picture

Issue tags: -Drupal 10 compatibility +Drupal 10 compatibility (2.x branch)

I am updating the title to distinguish this issue from #3293759: Drupal 10 compatibility (1.x branch).

benjifisher’s picture

Title: Drupal 10 compatibility » Drupal 10 compatibility (2.x branch)
Issue tags: -Drupal 10 compatibility (2.x branch) +Drupal 10 compatibility

Oops, I updated the tag instead of the title.

  • bircher committed 03f326b on 2.0.x
    Issue #3284552 by bircher, benjifisher: Drupal 10 compatibility
    
bircher’s picture

I guess we can open another issue in case this doesn't fix it even though tests pass now.

bircher’s picture

Status: Needs review » Fixed

  • bircher committed 5e37749 on 2.0.x
    Issue #3284552 by bircher, benjifisher: Drupal 10 compatibility
    

Status: Fixed » Closed (fixed)

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