Problem/Motivation

First step of #3215870: Require Composer 2.3.5 for developing Drupal 10 core.

Proposed resolution

Require at least composer/composer 2.2.4 as per current discussion. Leave it as a dev dependency now. It may be further increased to 2.3 in #3264918: Update symfony/console to Symfony 6.

Remaining tasks

Followup: #3265124: Update composer/installers to ^2.0

User interface changes

N/A

API changes

TBD

Data model changes

N/A

Release notes snippet

Needed.

Issue fork drupal-3265094

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
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

xjm created an issue. See original summary.

xjm opened merge request !1827

xjm’s picture

xjm
she/her
Primary language English
CreditAttribution: xjm at Acquia commented
Status: Active » Needs review
Issue tags: +Drupal 10
xjm’s picture

xjm
she/her
Primary language English
CreditAttribution: xjm at Acquia commented
Issue summary: View changes
Related issues: +#3264918: Update symfony/console to Symfony 6
longwave’s picture

longwave
he/him
Primary language English
Location UK
CreditAttribution: longwave at Full Fat Things commented

I think we should bump composer/installers to ^2.0. Reasoning:

We also include composer/installers in the recommended and legacy project templates. I guess we should bump it there too, but does that need a separate issue?

longwave’s picture

longwave
he/him
Primary language English
Location UK
CreditAttribution: longwave at Full Fat Things commented

There is also the interesting note

As of Composer 2.1, the Composer\InstalledVersions class has a getInstalledPackagesByType method which can let you figure out at runtime which plugins/modules/extensions are installed.

It's likely a long way off but in a Composer-only world we could perhaps use this for module discovery.

xjm’s picture

xjm
she/her
Primary language English
CreditAttribution: xjm at Acquia commented

@longwave, good points, thanks.

I checked with the Automatic Updates team and they don't currently interact with the dependency, so let's get this in first for that reason, and handle the composer/installers update as a followup.

xjm’s picture

xjm
she/her
Primary language English
CreditAttribution: xjm at Acquia commented

Posted #3265124: Update composer/installers to ^2.0 for the followup. Thanks!

xjm’s picture

xjm
she/her
Primary language English
CreditAttribution: xjm at Acquia commented
Issue summary: View changes
xjm’s picture

xjm
she/her
Primary language English
CreditAttribution: xjm at Acquia commented
Title: Update our Composer dev dependencies to D10 minima » Update our composer/composer dev dependency to the required minimum for Automatic Updates
xjm’s picture

xjm
she/her
Primary language English
CreditAttribution: xjm at Acquia commented
Issue summary: View changes
catch’s picture

catch
he/him
Primary language English
CreditAttribution: catch at Third and Grove commented
Status: Needs review » Reviewed & tested by the community

If we're going to bump composer/installers in a follow-up, I think this is ready to go.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott at Acro Commerce commented
Status: Reviewed & tested by the community » Fixed

Committed bedd3d0 and pushed to 10.0.x. Thanks!

  • alexpott committed bedd3d0 on 10.0.x 
    Issue #3265094 by xjm, longwave: Update our composer/composer dev...
xjm’s picture

xjm
she/her
Primary language English
CreditAttribution: xjm at Acquia commented
Issue tags: +10.0.0 release notes

Status: Fixed » Closed (fixed)

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

xjm closed merge request !1827