Problem/Motivation

Super short version: cweagans/composer-patches that we require is breaking DrupalCI and there's no way to add it to the allow list before DrupalCI does the composer install. For details, see #3307885: Customize DrupalCI config to allow running cweagans/composer-patches; fails otherwise

Steps to reproduce

Run DrupalCI, get Composer require error.

Proposed resolution

Re-assess which patches we need and whether a new version of a dependency can be required that includes those patches, eventually removing cweagans/composer-patches so that we can run DrupalCI.

Multiselect module

We currently require version 8.x-1.2 of this which is the latest of that branch; this doesn't contain the fixes we use patches for; the 2.x branch which (at time of writing) has a beta release does contain the fixes from the patches.

It doesn't look look like either issue is necessary for Config Enforce Devel to function, so we could expand the version constraint to add ^2.0 and remove the patches, and most or all users will get the newer version with the fixes while technically not breaking backwards compatibility with Drupal 8, which has been end of life for months now.

Configuration development module

This one is likely to be the blocker as it doesn't look like the maintainers are present much or at all going back two or three years. Neither of these are even merged, though there is recent activity:

We'll probably have to reach out the maintainers of the module to get these issues looked at once they're ready. We could also ask to be added as maintainers if they don't have time, maybe just temporarily to get these done and released, and then they can remove us as maintainers unless one or more of us have the bandwidth to help out in their issue queue.

Remaining tasks

Do the things.

User interface changes

Minimal or none, depending on Multiselect.

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

Ambient.Impact created an issue. See original summary.

ambient.impact’s picture

Status: Active » Postponed

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

  • spiderman committed 9fdf4f8c on 1.0.x
    Relates to #3343504: Remove patch to config_devel Drupal 10...
spiderman’s picture

I was finding a fresh build failing due to the D10 compatibility fix no longer applying, so I've pushed a commit on `1.0.x` to remove it. We may want to roll a new release on this basis, just to avoid composer failures downstream of us.

spiderman’s picture

@raystuart Thanks for the MR! I'm sorry I didn't recognize your patch was the same as what I was doing, or I would have just merged that. In any case, we ended up in the same place. Appreciate your input :)

raystuart’s picture

Oh, no worries @spiderman , it was easy to miss. I figured you probably committed and then remembered that this issue even exists.

ambient.impact’s picture

Marking as fixed; see the attached related issues.

  • spiderman committed 9fdf4f8c on 2.0.x
    Relates to #3343504: Remove patch to config_devel Drupal 10...

Status: Fixed » Closed (fixed)

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