Skip to content

馃敡 Only run unittests on pushes to main - #611

Merged
MiWeiss merged 2 commits into
mainfrom
ci/scope-push-trigger-to-main
Sep 2, 2026
Merged

MiWeiss merged 2 commits into
mainfrom
ci/scope-push-trigger-to-main

Conversation

@MiWeiss

@MiWeiss MiWeiss commented Sep 2, 2026

Copy link
Copy Markdown
Collaborator

push: in linting_and_unittests.yml has no branch filter, so every push to a PR branch queues the 15-job matrix twice: once for push, once for pull_request. Verified on the current queue, e.g. fix/block-hash has both a unittests/push and a unittests/pull_request run for the same commit.

Scoping push: to main matches what pre-commit.yml and codeql-analysis.yml already do, and halves CI load on every PR branch push. Pushes to main and all pull request runs are unaffected.


馃 Generated with Claude Code

`push:` had no branch filter, so every push to a PR branch triggered the
15-job matrix twice: once for `push`, once for `pull_request`. Scoping it
to main matches pre-commit.yml and codeql-analysis.yml and halves CI load
on PR branches.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@MiWeiss
MiWeiss force-pushed the ci/scope-push-trigger-to-main branch from f192b86 to ea4dc3f Compare September 2, 2026 20:38
@MiWeiss
MiWeiss merged commit c6c0011 into main Sep 2, 2026
16 checks passed
@MiWeiss
MiWeiss deleted the ci/scope-push-trigger-to-main branch September 10, 2026 20:04
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