release: v0.3.0-beta.2 - #161
Conversation
GSTACK-Checkpoint: 2026-07-14/e9-beta-draft-publish#1 Signed-off-by: Gnani Rahul <gnani.nutakki@gmail.com>
…aft-publish fix(e9): publish beta drafts by release ID
📝 WalkthroughWalkthroughThe beta release workflow now obtains the release database ID with ChangesBeta release publication repair
Estimated code review effort: 2 (Simple) | ~10 minutes Possibly related issues
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@tests/scripts/release_tag_policy_test.sh`:
- Line 53: Strengthen the assertion in the release workflow validation around
the grep for `gh release view` so it verifies scalar extraction via `--jq
'.databaseId'`, not just the `--json databaseId` field. Match the complete
command or add a separate assertion covering the jq expression.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro Plus
Run ID: cbb22021-5c99-49d4-8e9c-4945cd1aa33c
📒 Files selected for processing (3)
.github/workflows/release.ymlsessions/2026-07-14-e9-beta-draft-publish.mdtests/scripts/release_tag_policy_test.sh
|
|
||
| grep -Fq 'hack/release-tag-classify.sh "$GITHUB_REF_NAME"' "${workflow}" | ||
| printf '[release-tag-policy] PASS: workflow invokes the tested classifier\n' | ||
| grep -Fq 'gh release view "$GITHUB_REF_NAME" --json databaseId' "${workflow}" |
There was a problem hiding this comment.
🎯 Functional Correctness | 🟡 Minor | ⚡ Quick win
Assert scalar database ID extraction.
Line 53 checks only --json databaseId; it would still pass if --jq '.databaseId' were removed and the full JSON object were passed to the PATCH URL. Match the complete lookup or separately assert the --jq expression.
🧰 Tools
🪛 Shellcheck (0.11.0)
[info] 53-53: Expressions don't expand in single quotes, use double quotes for that.
(SC2016)
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@tests/scripts/release_tag_policy_test.sh` at line 53, Strengthen the
assertion in the release workflow validation around the grep for `gh release
view` so it verifies scalar extraction via `--jq '.databaseId'`, not just the
`--json databaseId` field. Match the complete command or add a separate
assertion covering the jq expression.
Corrected beta release boundary
Promotes the draft-safe beta publication fix from
devtomainfor a new immutablev0.3.0-beta.2tag.gh release view --json databaseIdprerelease=trueand API stringmake_latest=false; stable release behavior is unchangedEvidence
devCI:29368327885passeddevCodeQL:29368327085passedAfter this PR is independently green and merges, the exact main CI will run before creating a new signed
v0.3.0-beta.2tag.Summary by CodeRabbit
Bug Fixes
Tests
Documentation