Require explicit release publication opt-in - #144
Conversation
|
Warning Review limit reachedYou’ve reached a temporary PR review limit under our Fair Usage Limits Policy. Next review available in: 15 minutes Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available. How can I continue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews. How do review limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (3)
Comment |
b588cb9 to
1585abe
Compare
1585abe to
6b447e9
Compare
| local block=$1 | ||
| local line | ||
| while IFS= read -r line; do | ||
| if [[ "$line" =~ ^[[:space:]]+if:[[:space:]]*\$\{\{[[:space:]]*inputs\.publish[[:space:]]*==[[:space:]]*true[[:space:]]*\}\}[[:space:]]*$ ]]; then |
There was a problem hiding this comment.
This guard still requires one exact spelling of the publish condition. A workflow with the same boolean input defaulting to false and an equivalent positive gate such as if: ${{ inputs.publish }} remains explicitly opt-in, but this regex rejects it and blocks CI. The guard should check the publish job's safety property instead of requiring this single expression form.
There was a problem hiding this comment.
No change. The guard intentionally requires the explicit typed comparison from the release policy: inputs.publish == true. Shorthand truthiness is excluded so future edits cannot weaken the positive boolean opt-in. Field order and unrelated job settings remain flexible.
|
Latest commit passed Namespace CI run 29212802158, including the release-gate self-test. |
Summary
Validation
Greptile Summary
This PR makes release publication an explicit opt-in workflow action. The main changes are:
publishinput withdefault: falsein the release workflow.inputs.publish.Confidence Score: 4/5
This is close, but I would fix the remaining guard behavior before merging.
scripts/check-release-publish-gate.sh
Important Files Changed
Reviews (3): Last reviewed commit: "fix(ci): require explicit release public..." | Re-trigger Greptile