docs(github): clarify issue taxonomy#816
Conversation
|
Warning Rate limit exceeded
You’ve run out of usage credits. Purchase more in the billing tab. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (5)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Code Review
This pull request introduces a new tech-debt issue template and elevates tech-debt to a primary type label, decoupling it from the task label. It also updates the task template, triage documentation, and label policy scripts to reflect this change. The review feedback suggests reordering the label lists across the documentation and policy scripts to ensure consistency with the triage guide, where task is treated as a fallback. Additionally, the feedback recommends using explicit label names in template instructions and expanding the pull request template to include checks for target branches and Conventional Commits.
Summary
bug,enhancement,task, anddocumentation.tech-debtto pair withtask, so debt can be supplemental context on any primary type.taskfrom becoming the default agent-work bucket.Why
Closes #815.
The open issue backlog had 33 of 57 issues labeled
task, including UI polish, CI false positives, and internal debt. The problematic rule was not thattech-debtexisted, but that it had to be paired withtask, which encouraged usingtaskas a catch-all. This PR fixes that guardrail while preserving the PR #731 triage flow and PR #752 checklist discipline.Related Issue
Closes #815
Human Review Status
Pending
Review Focus
tech-debtshould remain supplemental, not a primary type label.tech-debtwith any one primary type without allowing it to replace the primary type.taskas the default agent-work bucket.Risk Notes
tech-debtno longer satisfies the required type label by itself, but it also no longer requirestask.taskis not changed by this PR because label descriptions are repository metadata, not files. It should be updated after merge or by a maintainer action.How To Verify
Screenshots or Recordings
Not required; GitHub template and policy text changes only, with no app UI surface.
Checklist
bug,enhancement,task,documentation. Type labels are author-added; the labeler bot does NOT assign them. Add the label in the GitHub UI, then tick this.app,ui,platform,harness,ci. The labeler bot assigns these on PR open based on changed paths. Confirm the bot's choice (or override if wrong), then tick this.P0,P1,P2,P3. The priority-triage bot suggests one on PR open. Confirm or override, then tick this.Pending,Approved by @<reviewer>, orNot required: <reason>(default isPending; "not required" is restricted to bot-authored low-risk PRs).dev, and my PR title and commit messages use Conventional Commits in English.