Set release repo context without checkout#109
Conversation
Review Summary by QodoSet GH_REPO context for no-checkout release workflow jobs
WalkthroughsDescription• Add GH_REPO environment variable to release workflow jobs • Fixes gh release upload/edit failures in no-checkout jobs • Add validation tests for GH_REPO context in release invariants Diagramflowchart LR
A["release.yml<br/>no-checkout jobs"] -->|"missing GH_REPO"| B["gh release upload/edit<br/>fails to infer repo"]
C["Add GH_REPO env var<br/>to release-assets-draft<br/>and publish-github-release"] -->|"explicit repo context"| D["gh commands<br/>work without checkout"]
E["release_signed_release_invariants.sh<br/>validation tests"] -->|"verify GH_REPO set"| F["ensure fix<br/>is maintained"]
File Changes1. .github/workflows/release.yml
|
Code Review by Qodo
1. Brittle GH_REPO invariant
|
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
There was a problem hiding this comment.
Code Review
This pull request updates the tests/release_signed_release_invariants.sh script to ensure that both the release-assets-draft and publish-github-release jobs explicitly define the GH_REPO environment variable. The reviewer noted that the regular expressions used to validate this environment variable have issues with inconsistent curly brace escaping, strict spacing, and missing support for optional double quotes. Robust regex suggestions were provided to address these portability and flexibility concerns.
Signed-off-by: Nelson Spence <nelson@projectnavi.ai>
602da8a to
3920e36
Compare
Summary:
Why:
Validation:
Recovery after merge: