Skip to content

Align setup-magento action with upstream defaults#355

Merged
damienwebdev merged 1 commit intomage-os:mainfrom
digitalrisedorset:ci/setup-magento-alignment
May 7, 2026
Merged

Align setup-magento action with upstream defaults#355
damienwebdev merged 1 commit intomage-os:mainfrom
digitalrisedorset:ci/setup-magento-alignment

Conversation

@digitalrisedorset
Copy link
Copy Markdown
Contributor

Use this:

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?

The setup-magento action diverged from the upstream implementation:

  • default PHP version was still set to 8.3
  • default Magento version was outdated
  • composer_auth input handling was missing
  • COMPOSER_AUTH was not passed to composer-related steps

Fixes: N/A

What is the new behavior?

Align setup-magento/action.yml with upstream defaults by:

  • bumping the default PHP version to 8.4
  • updating the default Magento version
  • adding composer_auth input support
  • forwarding COMPOSER_AUTH to composer-related steps

Does this PR introduce a breaking change?

  • Yes
  • No

Other information

This change assumes environments are not relying on Traefik-specific routing behaviour.

@digitalrisedorset digitalrisedorset requested a review from a team as a code owner May 7, 2026 14:06
Copy link
Copy Markdown
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

@damienwebdev damienwebdev merged commit 6e58a15 into mage-os:main May 7, 2026
1 check passed
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