Skip to content

ci: build every pull request, not only those based on main - #20

Merged
rasros merged 1 commit into
mainfrom
ci/build-stacked-prs
Aug 13, 2026
Merged

ci: build every pull request, not only those based on main#20
rasros merged 1 commit into
mainfrom
ci/build-stacked-prs

Conversation

@rasros

@rasros rasros commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

Summary

build.yml runs on every pull request rather than only those based on main.

Why

The workflow already ran on pull requests, so this is not about turning CI on; it is about which
pull requests it covers. pull_request: branches: [ main ] means a stacked pull request — one branch
based on another, targeting that other branch — gets no checks at all and can merge unverified.

klause hit this and dropped the filter; the change here is the same, comment included, so the repos
agree. push stays restricted to main, so this adds no duplicate runs beyond the stacked case it
is meant to cover.

Testing

The workflow parses, and its trigger now matches klause's byte for byte: pull_request present with
no base filter, push still [ main ].

@rasros
rasros force-pushed the ci/build-stacked-prs branch from 7b39d0a to dd9f868 Compare August 13, 2026 19:25
@rasros
rasros merged commit 6f37917 into main Aug 13, 2026
1 check passed
@rasros
rasros deleted the ci/build-stacked-prs branch August 13, 2026 19:31
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