Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: update PHP/Composer/Magento versions and defaults for actions #231

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

rhoerr
Copy link
Contributor

@rhoerr rhoerr commented Aug 7, 2024

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: Outdated versions of PHP/Composer/Magento referenced in actions

What is the new behavior?

  • All references to PHP 8.1 changed to 8.3
  • All references to Composer 2.2 changed to 2.7
  • All references to Magento 2.4.5 changed to 2.4.7-p2

This should bring everything up to date for latest releases.

Do not merge this until 2.4.7-p2 is tagged upstream and synced to our mirror repositories.

Does this PR introduce a breaking change?

  • Yes
  • No

Other information

@rhoerr rhoerr requested a review from a team as a code owner August 7, 2024 03:21
@rhoerr
Copy link
Contributor Author

rhoerr commented Aug 7, 2024

Do not merge this until 2.4.7-p2 is tagged upstream and synced to our mirror repositories, due to the version references.

php-version: 8.1
tools: composer:v2.4.2
php-version: 8.3
tools: composer:v2.7.7
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
tools: composer:v2.7.7
tools: composer:v2.8.4


- uses: ./get-composer-version
id: composer-version


- name: Fail if versions do not match
if: steps.composer-version.outputs.version != '2.4.2'
if: steps.composer-version.outputs.version != '2.7.7'
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
if: steps.composer-version.outputs.version != '2.7.7'
if: steps.composer-version.outputs.version != '2.8.4'

php-version: 8.1
tools: composer:v2.4.2
php-version: 8.3
tools: composer:v2.7.7
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
tools: composer:v2.7.7
tools: composer:v2.8.4


- run: composer create-project --repository-url="https://mirror.mage-os.org" "magento/project-community-edition:2.4.5-p1" ../magento2 --no-install
- run: composer create-project --repository-url="https://mirror.mage-os.org" "magento/project-community-edition:2.4.7-p2" ../magento2 --no-install
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
- run: composer create-project --repository-url="https://mirror.mage-os.org" "magento/project-community-edition:2.4.7-p2" ../magento2 --no-install
- run: composer create-project --repository-url="https://mirror.mage-os.org" "magento/project-community-edition:2.4.7-p3" ../magento2 --no-install

tools: composer:v2
mode: extension
magento_version: magento/project-community-edition:2.4.5-p1
magento_version: magento/project-community-edition:2.4.7-p2
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
magento_version: magento/project-community-edition:2.4.7-p2
magento_version: magento/project-community-edition:2.4.7-p3


- uses: mage-os/github-actions/cache-magento@main
with:
mode: 'extension'
composer_cache_key: 'magento/project-community-edition:2.4.5-p1'
composer_cache_key: 'magento/project-community-edition:2.4.7-p2'
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
composer_cache_key: 'magento/project-community-edition:2.4.7-p2'
composer_cache_key: 'magento/project-community-edition:2.4.7-p3'

@@ -69,7 +69,7 @@ runs:
- name: Check if allow-plugins option is available for this version of composer
uses: mage-os/github-actions/semver-compare@main
with:
version: 2.2
version: 2.7
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
version: 2.7
version: 2.8

tools: composer:v2
mode: extension
magento_version: 2.4.5-p1
magento_version: 2.4.7-p2
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
magento_version: 2.4.7-p2
magento_version: 2.4.7-p3

@@ -27,7 +27,7 @@ inputs:

magento_version:
required: false
default: '~2.4.5'
default: '~2.4.7-p2'
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
default: '~2.4.7-p2'
default: '~2.4.7-p3'

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