Skip to content

Drop pull_request trigger from version_command workflow#59

Merged
TomasVotruba merged 1 commit into
mainfrom
drop-version-command-pr-trigger
Jul 22, 2026
Merged

Drop pull_request trigger from version_command workflow#59
TomasVotruba merged 1 commit into
mainfrom
drop-version-command-pr-trigger

Conversation

@TomasVotruba

Copy link
Copy Markdown
Member

Removes the pull_request trigger from the version_command workflow.

Why

The job runs composer require symplify/easy-coding-standard from Packagist, then asserts ecs --version prints the PHP-CS-Fixer line — a smoke test of the published scoped+downgraded artifact (guards #8).

On pull_request it installs the currently-published package, not the PR code — so the result is identical regardless of the diff. It never tests the change, yet the fresh, uncached composer require makes it one of the slowest jobs on every PR.

The push (main + tags) and weekly schedule triggers already cover the published artifact, which is the only thing this test can actually catch drift in.

Change

 on:
-    pull_request:
     push:
         branches:
             - main
         tags:
             - '*'
     schedule:
         -   cron: '0 6 * * 1'

@TomasVotruba
TomasVotruba merged commit 5de591d into main Jul 22, 2026
7 checks passed
@TomasVotruba
TomasVotruba deleted the drop-version-command-pr-trigger branch July 22, 2026 15:46
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