Skip to content

Conversation

@rhoerr
Copy link
Contributor

@rhoerr rhoerr commented Dec 30, 2025

PR Checklist

Please check if your PR fulfills the following requirements:

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Documentation content changes
  • Other... Please describe:

What is the current behavior?

Fixes: Some past Magento releases in supported-versions had unpinned composer: 2 constraints, causing it to use the latest Composer 2 version for those. Those past versions are not compatible with composer 2.9.

This caused ~half of the errors in this mirror preview build: https://github.com/mage-os/generate-mirror-repo-js/actions/runs/20271742450

This fixes resulting integrity test errors by pinning them to the composer versions of the time -- 2.2 for all of the early versions in question.

What is the new behavior?

Installations will run for those versions with composer 2.2 instead of 2.9, avoiding the incompatible security audit introduced in composer 2.9.

I also set the composer version to 2.9.2 for the 'next' version (which is used only for nightlies).

Does this PR introduce a breaking change?

  • Yes
  • No

Other information

Tests pass:

Test Suites: 10 passed, 10 total
Tests:       69 passed, 69 total
Snapshots:   0 total
Time:        1.51 s
Ran all test suites.

@rhoerr rhoerr requested a review from damienwebdev December 30, 2025 01:32
@rhoerr rhoerr requested a review from a team as a code owner December 30, 2025 01:32
Copy link
Member

@damienwebdev damienwebdev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@rhoerr rhoerr merged commit 2c91044 into mage-os:main Dec 30, 2025
118 of 141 checks passed
@rhoerr rhoerr deleted the fix/composer-constraints branch December 30, 2025 01:36
@mage-os-ci mage-os-ci mentioned this pull request Dec 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants