Repository navigation
Fix release candidate validation while preserving Copilot ownership - #163
Conversation
|
To preview the documentation for this pull request, visit the following URL: docs.page/vypdev/zstandard~163
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## develop #163 +/- ##
============================================
+ Coverage 54.10% 77.36% +23.26%
============================================
Files 12 14 +2
Lines 573 1109 +536
============================================
+ Hits 310 858 +548
+ Misses 263 251 -12
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
Bugbot: review complete
|
Copilot approval · Human review neededStatus: A protected workflow, policy, or trust-boundary file changed. Technical evidenceRevision: |
There was a problem hiding this comment.
Current status: All findings originating in this review are resolved.
Last reconciled on59369d9. See aggregate Bugbot status.
🤖 Bugbot review snapshot
Bugbot reported 1 potential problem when commit fac7e9d was analyzed. This snapshot is historical; use the status block above for current state. 1 finding is linked to changed code.
Findings
- medium: Release tag validation ignores the configured production branch —
scripts/check_release_tag.sh:17
There was a problem hiding this comment.
Current status: All findings originating in this review are resolved.
Last reconciled on59369d9. See aggregate Bugbot status.
🤖 Bugbot review snapshot
Bugbot reported 1 potential problem when commit acf1d54 was analyzed. This snapshot is historical; use the status block above for current state. 1 finding is linked to changed code.
Findings
- medium: Pin the newly added checkout action to a commit SHA —
.github/workflows/release_workflow.yml:790
Release preparation could fail to resolve the Linux ARM64 toolchain or retry unchanged native outputs, and publication ended outside Copilot's deployment lifecycle. This change builds and tests one exact candidate, then requires accepted production content and successful preparation evidence before package publication.
Closes #162.
Changes
MAIN_BRANCHandRELEASE_TREEconsistently. Real Git tests reject tags accepted only on an unrelated default branch and preserve the GitHub-only hotfix lane.Copilot's configuration, guidance, internal workflows, verification setup, hotfix workflow and
vypdev/copilot@v3references are unchanged. Package versions are unchanged.Validation for
59369d9Both Android integration files also pass local Dart analysis and formatting. Shell syntax and actionlint checks pass. actionlint 1.7.12 uses a narrow exception for its unsupported
concurrency.queueparser rule; the installed Copilot queue contract is retained.Publication readiness
On 2026-10-25 the maintainer confirmed GitHub-only automated publishing for all ten packages: repository
vypdev/zstandard, tag patternv{{version}}, required environmentpub.dev. GitHub environment approval and branch/tag protections were configured separately. This is configuration evidence; no live OIDC publishing operation has been run.No merge, repository version tag, GitHub release or package publication has been performed. A future release still requires the explicitly authorized Copilot deployment operation and environment approval.