ci: opt-in Blender 5.1 smoke via needs-5.1 and dispatch - #137
Merged
Conversation
Default PR matrix stays 4.5 and 5.2. labeled is required so the label actually starts a job. Dispatch takes a series input. Monday cron is unchanged. Signed-off-by: TMHSDigital <154358121+TMHSDigital@users.noreply.github.com> Co-authored-by: Cursor <cursoragent@cursor.com>
Place the how-to in CONTRIBUTING.md, where version targeting already lives, and point at it from the workflow header, README, AGENTS.md, and the PR template. Signed-off-by: TMHSDigital <154358121+TMHSDigital@users.noreply.github.com> Co-authored-by: Cursor <cursoragent@cursor.com>
Temporary SystemExit(1) in bmesh-gear so the labeled 5.1 job is observed failing. Reverted in the next commit. Signed-off-by: TMHSDigital <154358121+TMHSDigital@users.noreply.github.com> Co-authored-by: Cursor <cursoragent@cursor.com>
This reverts commit 0d32dbd. Signed-off-by: TMHSDigital <154358121+TMHSDigital@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Closes #134. Opt-in 5.1 smoke without putting 5.1 on every PR.
This repo has no
ci.yml. The existing lever isblender-smoke.yml's emptyworkflow_dispatch: {}. This PR adds aserieschoice input there, plus aneeds-5.1label path.pull_request.typeswas previously unset (GitHub default: opened / synchronize / reopened only).labeledis now listed explicitly so applying the label starts a run. Monday cron is unchanged. Default PR matrix stays 4.5 and 5.2.Auto-label does not apply
needs-5.1. It still only applies path labels (ci,documentation, ...). GITHUB_TOKEN label events do not retrigger workflows; a human (or REST) applyingneeds-5.1does.B2 gate-failure proof (live-run-proven)
needs-5.1is left applied on this PR so the final head keeps a 5.1 job.1. Absent by default (no
needs-5.1)gh pr checks 137immediately after open, before the label. NoBlender 5.1 smokerow. Smoke run: https://github.com/TMHSDigital/Blender-Developer-Tools/actions/runs/34691306483 (jobs: Resolve smoke matrix, Blender 4.5, Blender 5.2 only).That same run later finished 4.5 pass 3m10s / 5.2 pass 4m27s. Still no 5.1 job.
2. Triggers on label
Applied
needs-5.1via REST (gh pr edit --add-labelhit the projectCards GraphQL deprecation). Labeled run matrix log:Smoke matrix: ["5.1"]. 5.1 job appeared and completed.Labeled run: https://github.com/TMHSDigital/Blender-Developer-Tools/actions/runs/34691361129
3. Canary turns 5.1 red, then revert is green
Canary:
SystemExit(1)at the start ofexamples/bmesh-gearmain()(0d32dbd). Synchronize with the label on, matrix all three series.Failing run: https://github.com/TMHSDigital/Blender-Developer-Tools/actions/runs/34691589680
Revert
dc8072c. Green run: https://github.com/TMHSDigital/Blender-Developer-Tools/actions/runs/34691713403Documentation (inspection)
One how-to section in
CONTRIBUTING.mdafter Blender Version Targeting (that is where contributors already look for version policy). Pointers: workflow file header, README 5.1 table row + examples blurb, AGENTS.md CI/CD bullet, PR template evidence line.Evidence labels
ci.ymlin this repo; auto-label allow-list; Monday cron cron expression unchanged; default matrix still 4.5+5.2 on unlabeled PRsOut of scope
Agent bridge. LICENSE / #131. Baking/UV (Phase 5). VERSION / CHANGELOG / release.yml. Monday cron schedule and default PR version matrix. Examples/skills/snippets/rules/templates except the reverted canary. Exit-code cleanup is #136, not this PR.