Repository navigation
docs(contributing): drop the transitional blank-issue fallback - #4959
Conversation
The issue forms reach main with #4939, and config.yml sets blank_issues_enabled: false, so "until then, open a blank issue" describes a state that no longer exists and recommends a fallback that is unavailable. State the steady-state rule instead: a change to the forms goes live at the next milestone merge. Closes #4940 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Signed-off-by: cliffhall <cliff@futurescale.com>
|
There was a problem hiding this comment.
Copilot review overview
🟢 Approval recommended
The focused documentation update matches the issue requirements and existing issue-template guidance.
Review effort: Balanced
Findings: None
What changed in this PR
Updates contribution guidance to reflect the steady-state issue-form workflow after #4939.
Changes:
- Removes the unavailable blank-issue fallback.
- Clarifies when issue-form changes become active.
| File | Description |
|---|---|
CONTRIBUTING.md |
Documents that form changes go live with the next milestone merge. |
💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.
Takes CONTRIBUTING.md from v2/main, which dropped the transitional blank-issue fallback (#4940), so this branch stays byte-identical to v2/main and main does not receive the stale sentence. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Signed-off-by: cliffhall <cliff@futurescale.com>
Closes #4940
Description
Drops the transitional sentence in
CONTRIBUTING.md("How to report a bug or request a feature") that told reporters to open a blank issue until the forms reachedmain. Once #4939 merges, the forms are in the chooser, and.github/ISSUE_TEMPLATE/config.ymlsetsblank_issues_enabled: false, so the sentence describes a state that no longer exists and recommends a fallback that is unavailable.The paragraph now states the steady-state rule, matching the header comments in the three
.github/ISSUE_TEMPLATE/*.ymlfiles: a change to the forms goes live at the next milestone merge, not when it merges intov2/main.Sequencing: once this merges,
CONTRIBUTING.mdon #4939's branch is refreshed fromv2/main, so that PR stays byte-identical tov2/mainand what reachesmainahead of v2.0.0 does not carry the stale sentence.Server Details
Motivation and Context
Raised by Copilot on #4939 (#4939 (comment)), declined there because that PR must match
v2/main. See #4940.How Has This Been Tested?
No client-observable surface; targeted probes:
grep -n -i "blank issue\|until then" CONTRIBUTING.md .github/ISSUE_TEMPLATE/* .github/pull_request_template.mdfinds only theconfig.ymlcomment explaining that blank issues are disabled.npx prettier --check CONTRIBUTING.mdpasses.npm run local:gatewas not run: no code, package, workflow or skill is touched.Breaking Changes
None.
Types of changes
Checklist
🤖 Generated with Claude Code