ci: fractional platform validation and exact-SHA release preflight - #272
Conversation
|
Navigate logical layers of code changes, visualize relationships, and explore their blast radius. Note Currently processing new changes in this PR. This may take a few minutes, please wait... ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (49)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
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. Comment |
|
Release-safety review note: this is a useful pretag gate, but it is an interim step rather than the fleet-normalized issue-driven release mechanism in soldr #3343/#3345. Both manual publishers currently require candidate_sha == GITHUB_SHA at workflow_dispatch ( |
Scope
Implements fractional CI for FastLED/cli. Ordinary PR/main runs retain Linux x86 lint;
ci-testadds host tests;ci-fullselects all 29 declared platform workflows, including native Windows ARM tests. The full-coverage report requires successful platform test steps on one exact commit SHA.Both release entry points are manual and fail closed before creating a tag:
auto-release.ymlverifies a dispatch full-coverage report, builds six wheel/binary pairs, and checks artifacts.dry_run=trueperforms the preflight without tagging or publishing.vscode-extension-release.ymlverifies the same exact-SHA full-coverage report, builds seven VSIX packages, and checks exact version, target coverage, ZIP integrity, and extension metadata before creatingvscode-v<version>. It also supportsdry_run=true. Pushing a VS Code tag no longer starts publication.The candidate SHA must be the dispatch branch head. A conflicting existing tag fails. The normalized issue-driven release front door and a live nonpublishing release dry run remain pending; registry availability cannot be guaranteed by artifact lint.
Proof and rollout state
The stacked Intel macOS PR #273 passed all 29 platform workflows and the full-coverage sentinel at its earlier head
36221cf(run 35914920276). It also produced a same-SHA unlabeled event for runner-minute measurement. Since this PR has now advanced to0b38e4eto close the VS Code tag gap, #273 must include the new base commit and rerun full coverage at its new head before merge.The VS Code release safety change was tested RED→GREEN: both new tests failed before implementation; afterward all 17 focused CI-mode tests passed. Ruff, YAML structure parsing, and
git diff --checkpassed. This PR remains draft under zackees/soldr#3345.