Document bounty preflight quality gate#429
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughCONTRIBUTING.md now includes a "Preflight Checks" section instructing contributors to run a local submission quality gate script before opening bounty PRs. The section documents the script command and the validations it performs. ChangesSubmission Preflight Guidance
Possibly related PRs
🚥 Pre-merge checks | ✅ 5 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. Comment |
prettyboyvic
left a comment
There was a problem hiding this comment.
Reviewed PR #429 at head b6de214 against origin/main for Bounty #426.
No blockers found. The diff is limited to CONTRIBUTING.md and adds a contributor-facing preflight section that matches the existing scripts/submission_quality_gate.py workflow: bounty reference, summary/evidence signals, payability/capacity, attempts, similar PRs, and maintainer activity when public context is available. The copy also avoids payout, price, off-ramp, or acceptance claims.
Validation performed:
git show --stat origin/main..origin/pr/429-> 1 file changed, 15 insertions inCONTRIBUTING.md./.venv/Scripts/python.exe scripts/docs_smoke.py-> docs smoke okgit diff --check origin/main...HEAD-> clean
aiautotool
left a comment
There was a problem hiding this comment.
Reviewed PR #429 against current origin/main after fetching the latest base. The PR is now a focused docs-only change: 15 lines added to CONTRIBUTING.md for the advisory submission quality gate.
I checked that the new contributor-facing instructions match the existing tool and docs model: the command points to scripts/submission_quality_gate.py --text-file ... --repo ..., and the text accurately describes advisory checks for bounty reference, summary/evidence, award capacity, active attempts, similar PRs, and maintainer activity when public context is available. It does not claim the gate reserves work, guarantees acceptance, or creates payment.
Local verification on head b6de214f4b229e4caf224efcbd2dfd63435bdafc:
git diff origin/main...HEAD -- CONTRIBUTING.md-> only the preflight section changedpython scripts/docs_smoke.py-> docs smoke okgit diff --check origin/main...HEAD-> passed
No blocking issues found from my review.
|
Holding this out of the accepted batch. The PR is still not mergeable in the queue because the required project CI is not present/green on the current head. Review claims that did not catch that process blocker are not accepted for payment. |
|
I opened a rebased replacement at #450 because this PR head is not writable from the currently available automation account, and the maintainer noted this head is missing the required project CI signal. The replacement keeps the same focused |
Summary
CONTRIBUTING.mdfor bounty PR authors.scripts/submission_quality_gate.py --text-file ... --repo ...workflow and what the gate checks.Refs #426
Evidence
scripts/submission_quality_gate.py, which checks bounty references, summary/evidence signals, payability, active attempts, similar open PRs, and maintainer activity when available.docs/agent-guide.mdalready documents this gate for agents; this PR adds the same contributor-facing preflight guidance to the main contributor entrypoint.CONTRIBUTING.mdonly.Test Evidence
python scripts/docs_smoke.py(docs, template, example, or onboarding changes)Result:
docs smoke okMRWK
Related bounty or issue (
Bounty #NorRefs #Nfor multi-award bounties): Refs #426Summary by CodeRabbit