Skip to content

fix(ci): ignore platform reqs in release & create-zip workflows#173

Closed
GytisZum wants to merge 1 commit into
mainfrom
fix/release-zip-ignore-platform-reqs
Closed

fix(ci): ignore platform reqs in release & create-zip workflows#173
GytisZum wants to merge 1 commit into
mainfrom
fix/release-zip-ignore-platform-reqs

Conversation

@GytisZum

Copy link
Copy Markdown
Collaborator

Problem

The latest v3.3.1 release run failed — the release workflow errored at the first composer install step, so the module ZIP was never built or attached to the release.

composer.json pins a fake platform PHP version:

"config": { "platform": { "php": "5.6" } }

All locked dependencies require PHP ≥ 7.1/8.x, so Composer's lock-file compatibility check fails immediately:

Your lock file does not contain a compatible set of packages. Please run composer update.
  - doctrine/deprecations 1.1.6 requires php ^7.1 || ^8.0 -> your php version (5.6; overridden via config.platform...)

This exact fix (--ignore-platform-reqs) was already applied to index.yml in commit 7375297, but never propagated to release.yml or create-zip.yml.

Fix

Add --ignore-platform-reqs to the composer install and composer dump-autoload calls in both release.yml and create-zip.yml, matching index.yml.

🤖 Generated with Claude Code

The release and create-zip workflows ran a bare composer install while
composer.json pins config.platform.php to 5.6. Since all locked deps
require PHP >= 7.1/8.x, composer's lock compatibility check failed
immediately and the ZIP was never built (latest v3.3.1 release run failed).

Add --ignore-platform-reqs to composer install/dump-autoload, matching
the fix already applied to index.yml.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@GytisZum GytisZum closed this Jun 30, 2026
@GytisZum GytisZum deleted the fix/release-zip-ignore-platform-reqs branch June 30, 2026 14:14
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.

1 participant