Skip to content

docs: require issue discussion before PR for external contributors - #1538

Merged
v1r3n merged 1 commit into
mainfrom
docs/issue-first-contribution-policy
Aug 14, 2026
Merged

docs: require issue discussion before PR for external contributors#1538
v1r3n merged 1 commit into
mainfrom
docs/issue-first-contribution-policy

Conversation

@nthmost-orkes

Copy link
Copy Markdown
Contributor

Summary

  • Adds an explicit Issue-first policy subsection to CONTRIBUTING.md
  • Makes clear that external contributors must file and discuss an issue with maintainers before writing any code
  • States that PRs submitted without prior issue discussion will be closed at the maintainers' discretion
  • Clarifies that simply commenting "I'll take this" on an issue is not sufficient — contributors must wait for maintainer confirmation of the approach

Why now

We've been receiving PRs from outside contributors that land without any prior discussion, requiring significant back-and-forth or outright closure. This change sets the expectation up front so contributors invest time in the right direction.

User impact

Outside contributors will see clearly what's expected before opening a PR, reducing wasted effort on both sides.

@mp-orkes mp-orkes left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 👍

@v1r3n
v1r3n merged commit 6c62d12 into main Aug 14, 2026
11 checks passed
@v1r3n
v1r3n deleted the docs/issue-first-contribution-policy branch August 14, 2026 03:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants