Workflows: check-extension.yaml + _internal_check_extension.yaml
Status: ➕ Graycore-only — evaluate for porting
Files: .github/workflows/check-extension.yaml, .github/workflows/_internal_check_extension.yaml
PR context
The graycore remove_cache_arg branch modifies check-extension.yaml:
- Removes
COMPOSER_MIRROR_PATH_REPOS: 1 from composer install steps
- Replaces the
coding-standard action call with inlined steps (get-composer-version → semver-compare → allow-plugins → composer require → phpcs config-set → phpcs run) — this brings it in line with how graycore is removing the built-in PHP-setup from the coding-standard action
- Removes the release-please guard from
_internal_check_extension.yaml
What these workflows do
check-extension.yaml is a reusable workflow that consumers call to check a Magento extension: runs coding standard, integration tests, and verifies installability. _internal_check_extension.yaml is the internal self-test.
Dependents
No mage-os dependents yet. In graycore, external repos call check-extension.yaml directly.
Decision required
Should these workflows be ported to the mage-os fork?
Arguments for porting:
- Provides a complete, batteries-included experience for Mage-OS extension authors
- Mirrors the value proposition of the graycore library
Arguments against porting:
- Mage-OS fork has its own equivalent workflows (
full-integration-tests.yaml, nx-integration-tests.yml)
- May require adaptation for Mage-OS-specific tooling (warden, nx)
Changes needed if porting (based on graycore remove_cache_arg target state)
- Copy
check-extension.yaml from graycore remove_cache_arg branch (not main — get the updated version with inlined coding standard steps and no COMPOSER_MIRROR_PATH_REPOS)
- Copy
_internal_check_extension.yaml from graycore remove_cache_arg branch (release-please guard already removed)
- Update all action refs from
graycoreio/github-actions-magento2/...@main to mage-os/github-actions/...@main
- Update token secret names to mage-os equivalents (e.g.
MAGE_OS_CI_TOKEN)
- Verify dependencies —
get-composer-version, semver-compare, setup-magento, integration.yaml must be updated per their respective plans before enabling this workflow
Workflows: check-extension.yaml + _internal_check_extension.yaml
Status: ➕ Graycore-only — evaluate for porting
Files:
.github/workflows/check-extension.yaml,.github/workflows/_internal_check_extension.yamlPR context
The graycore
remove_cache_argbranch modifiescheck-extension.yaml:COMPOSER_MIRROR_PATH_REPOS: 1from composer install stepscoding-standardaction call with inlined steps (get-composer-version → semver-compare → allow-plugins → composer require → phpcs config-set → phpcs run) — this brings it in line with how graycore is removing the built-in PHP-setup from the coding-standard action_internal_check_extension.yamlWhat these workflows do
check-extension.yamlis a reusable workflow that consumers call to check a Magento extension: runs coding standard, integration tests, and verifies installability._internal_check_extension.yamlis the internal self-test.Dependents
No mage-os dependents yet. In graycore, external repos call
check-extension.yamldirectly.Decision required
Should these workflows be ported to the mage-os fork?
Arguments for porting:
Arguments against porting:
full-integration-tests.yaml,nx-integration-tests.yml)Changes needed if porting (based on graycore
remove_cache_argtarget state)check-extension.yamlfrom graycoreremove_cache_argbranch (notmain— get the updated version with inlined coding standard steps and noCOMPOSER_MIRROR_PATH_REPOS)_internal_check_extension.yamlfrom graycoreremove_cache_argbranch (release-please guard already removed)graycoreio/github-actions-magento2/...@maintomage-os/github-actions/...@mainMAGE_OS_CI_TOKEN)get-composer-version,semver-compare,setup-magento,integration.yamlmust be updated per their respective plans before enabling this workflow