[Helias]

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
  • The core build workflow now runs automatically when changes are pushed to the master branch.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: 0910c73d-2f78-4f83-87b5-33567d8ea3ca

📥 Commits

Reviewing files that changed from the base of the PR and between 98663b0 and 9abd221.

📒 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 triggers when code is pushed to the master branch.

Changes

Workflow Trigger

Layer / File(s)	Summary
Master push filter .github/workflows/core-build.yml	The push trigger now includes the master branch.

Priority: ⬇️ Low

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

Change: Bug fix

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Description check	⚠️ Warning	The description explains the CI change and its impact, but it does not use the required template sections. It omits the proposed changes, issues addressed, source, tests performed, and step-by-step te…	Update the description with the required headings. Add the changes, issue reference or state that none applies, source if available, tests performed, and detailed steps to test the workflow change.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly identifies the CI change and its purpose: building once per pull request.
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.

Full details: Description check

Explanation

The description explains the CI change and its impact, but it does not use the required template sections. It omits the proposed changes, issues addressed, source, tests performed, and step-by-step testing instructions.

• Fix all pre-merge checks with AI

✨ 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

• X
• Mastodon
• Reddit
• LinkedIn

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