ci: run push workflows on main - #34
dv-picknik wants to merge 1 commit into
Conversation
The push triggers name master, which here mirrors upstream, while PRs merge to main, the default branch. No workflow runs on merge to main, so the pixi Windows crash introduced by #30 never showed there. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
Navigate logical layers of code changes, visualize relationships, and explore their blast radius. No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Team Run ID: 📒 Files selected for processing (6)
Included review availability: This review used your included allowance. 4 included reviews remain after this review. Your included PR review attempts over the past 7 days set your current allowance at 6 reviews per hour. 📝 SummarySummary by CodeRabbit
WalkthroughSix GitHub Actions workflows now run on pushes to ChangesWorkflow push triggers
Priority: ➖ Normal Merge Risk: ⚪ Minimal · up to This change directs the six workflows to run on pushes to main while leaving pull-request triggers unchanged. No concrete merge risk remains. Caution Pre-merge checks failedPlease resolve all errors before merging. Addressing warnings is optional.
❌ Failed checks (1 error)
✅ Passed checks (3 passed)
Full details: Human Review CheckExplanation The PR changes six GitHub Actions workflows, so it is a CI/CD change. Each workflow changes its push trigger from
Comment ✨ Finishing Touches 💡 1🛠️ Fix failing CI checks 💡
|
[written by AI]
Point the
pushtriggers of six workflows atmain. They namedmaster, which in this fork mirrors upstream, while PRs merge tomain, the default branch. So no workflow runs on merge tomain. That's how the pixi Windows crash from #30 went unnoticed there. #33 fixes the crash.doxygen-gh-pages.ymlalready listsmainand is unchanged. Note thatcmake_ubuntu.ymlwill also upload coverage to Codecov on every push tomain, and every job here runs once more per merge.pixi.yamlhas CRLF line endings, which the edit keeps.This was part of #33. CodeRabbit's Human Review Check fails any workflow change until a requested human reviewer overrides it, so this PR needs that override.
🤖 Generated with Claude Code