docs: align CONTRIBUTING patch coverage with codecov.yml (99%) - #2372
docs: align CONTRIBUTING patch coverage with codecov.yml (99%)#2372bohdansolovie wants to merge 2 commits into
Conversation
8337984 to
b0d322a
Compare
The hard gate was raised in JSONbored#1975; contributor docs still said 97%. Co-authored-by: Cursor <cursoragent@cursor.com>
b0d322a to
d13c625
Compare
|
Warning 🟨🟨🟨🟨🟨🟨🟨🟨🟨🟨🟨🟨 ⏸️ Gittensory review result - manual review recommendedReview updated: 2026-07-02 04:11:44 UTC
⏸️ Suggested Action - Manual Review Review summary Nits — 3 non-blocking
Review context
Contributor next steps
Signal definitions
🟩 Safe / merged · 🟦 Advisory · 🟨 Held for review · 🟥 Blocked / closed 💰 Earn for open-source contributions like this. Gittensor lets GitHub contributors earn for the work they already do — register to start earning →. Checked by Gittensory, a quiet PR intelligence layer for OSS maintainers.
|
Update stale 97% contributor guidance and lock it with a codecov-policy test so docs stay aligned with the 99% patch gate from JSONbored#1975. Co-authored-by: Cursor <cursoragent@cursor.com>
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #2372 +/- ##
=======================================
Coverage 95.71% 95.71%
=======================================
Files 222 222
Lines 24661 24661
Branches 8949 8949
=======================================
Hits 23605 23605
Misses 433 433
Partials 623 623 🚀 New features to boost your workflow:
|
JSONbored
left a comment
There was a problem hiding this comment.
Fix this, then resubmit:
The main maintainability detail is that the new test still hard-codes the same target string it reads from config instead of deriving the expected documentation text from that value.
Summary
codecov.yml(ci(codecov): raise patch coverage target #1975).Test plan