Skip to content

ci: build once per pull request - #28

Merged
Helias merged 1 commit into
masterfrom
ci/build-trigger
Sep 21, 2026
Merged

Helias merged 1 commit into
masterfrom
ci/build-trigger

Conversation

@Helias

@Helias Helias commented Sep 20, 2026 •

Copy link
Copy Markdown
Member

The push trigger had no branch filter, so a branch pushed in this repo built once on the push and again when the pull request opened, an hour of runner time each. Limits it to master, like most other modules. Pull requests from forks were never affected.

Summary by CodeRabbit

  • Chores
    • Limited the core build workflow’s push-triggered runs to the master branch.

@coderabbitai

coderabbitai Bot commented Sep 20, 2026 •

Copy link
Copy Markdown

Review Change StackReview Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: b45bb198-9624-4d62-8bac-00f87a7d537a

📥 Commits

Reviewing files that changed from the base of the PR and between 9c25df7 and e2bbc9e.

📒 Files selected for processing (1)
  • .github/workflows/core-build.yml

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.


📝 Walkthrough

Walkthrough

The core-build workflow now runs for pushes to master only. Pushes to other branches no longer trigger this workflow.

Changes

Core Build Trigger

Layer / File(s) Summary
Restrict push triggers
.github/workflows/core-build.yml
The push trigger now includes a master branch filter. Previously, pushes to any branch triggered the workflow.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~2 minutes

Change: Bug fix

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly summarizes the main change: CI builds once per pull request by restricting push triggers to the master branch.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@Helias
Helias merged commit 9d7b6c0 into master Sep 21, 2026
1 of 2 checks passed
@Helias
Helias deleted the ci/build-trigger branch September 21, 2026 06:32
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