← ClaudeAtlas

discuss-in-issue-firstlisted

For a non-trivial change, open or comment on the tracking issue and get the approach discussed BEFORE opening the PR — to make the change land FASTER, not to make it more likely to be accepted. Measured on Apache Solr: PRs with prior issue discussion merged at a median 3.4 days vs 5.8 days for cold PRs (~1.7× faster, p<0.001), with NO difference in merge rate (both ~83%). So discuss first to move faster, not to get in. Use before opening a PR for anything beyond a trivial typo/dep-bump. Trigger terms: issue-first, discuss first, open an issue, cold PR, RFC, design before code, faster merge.
serhiy-bzhezytskyy/contrib-receipts · ★ 0 · Code & Development · score 75
Install: claude install-skill serhiy-bzhezytskyy/contrib-receipts
# Discuss in the issue first (to move faster) ## Purpose Prior discussion on the tracking issue means the PR lands into pre-built consensus: the community already agreed the change matters and roughly how it should look, so review is cheaper and faster. The honest, measured claim is about **speed, not acceptance** — issue-first PRs merge markedly faster, but they do not merge at a higher *rate*. So the reason to do it is to shorten time-to-merge and avoid a rewrite after a cold PR meets a design objection, not to improve your odds of getting in at all. Calibration matters here: do NOT sell this as "gets accepted more often" (the data refutes that), and do NOT pair it with a "minimize review cycles" rule — in the same corpus, *more* review comments correlated with a *higher* merge rate, so cycles are a sign of engagement that lands, not of doom. ## When to use - Before opening a PR for any non-trivial change (behavior change, new API, refactor, anything with a design decision). - Skip for genuinely trivial changes — typos, broken links, mechanical dep-bumps. ## When NOT to use The change is genuinely trivial (typo, broken link, mechanical dep-bump) — the discussion overhead then costs more time than it saves. ## The practice (checklist) - [ ] Find or open the tracking issue for the change. - [ ] State the problem and your proposed approach on the issue; surface the design question rather than presenting a finished PR as a fait accompli. - [ ] Get a maintainer'