Skip to content

Update the CONTRIBUTING.md to build quality in the recent PRs - #2301

Merged
t-lo merged 4 commits into
mainfrom
update-contributing.md
Sep 1, 2026
Merged

Update the CONTRIBUTING.md to build quality in the recent PRs#2301
t-lo merged 4 commits into
mainfrom
update-contributing.md

Conversation

@sayanchowdhury

@sayanchowdhury sayanchowdhury commented Aug 7, 2026

Copy link
Copy Markdown
Member

Update the CONTRIBUTING.md to build quality in the recent PRs

AI has been used to grammar check the change and sentence restructing whereever required, and review em dashes. I've read through the suggested change and updated whereever required

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Updates CONTRIBUTING.md with additional contributor and maintainer guidance aimed at reducing low-effort / spammy issues and PRs, including explicit expectations for AI-assisted submissions and clearer issue-triage norms.

Changes:

  • Adds new guidance sections covering AI-assisted contributions, PR etiquette, issue assignment expectations, issue quality requirements, and how to report spam/abuse/security issues.
  • Expands “Finding Issues” guidance to require a short, contributor-written plan (and discourages AI-generated “claim” comments).
  • Cleans up a few formatting/details in the commit message guidelines examples and rules table.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread CONTRIBUTING.md Outdated
Comment thread CONTRIBUTING.md Outdated
Copilot AI review requested due to automatic review settings August 7, 2026 09:01
@sayanchowdhury
sayanchowdhury force-pushed the update-contributing.md branch from 967911c to 7f4936c Compare August 7, 2026 09:02

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

Suppressed comments (1)

CONTRIBUTING.md:95

  • The PR description says "no-em dashes", but this newly added text introduces an em dash (—). Either update the PR description to reflect current formatting intent, or adjust the new text to avoid em dashes for consistency with the stated goal.
Found something you'd like to work on? Excellent! Just leave a comment like
"I'd like to work on this" — that's all it takes to claim it.

Copilot AI review requested due to automatic review settings August 7, 2026 09:04

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

Suppressed comments (1)

CONTRIBUTING.md:95

  • This implies that a simple “I’d like to work on this” comment is enough to claim an issue, but the new “Issue Assignment” section later explicitly says contributors should include an intended approach and that issues won’t be formally assigned until there’s a draft PR. This is internally inconsistent guidance and will confuse new contributors.
Found something you'd like to work on? Excellent! Just leave a comment like
"I'd like to work on this" — that's all it takes to claim it.

Comment thread CONTRIBUTING.md
Copilot AI review requested due to automatic review settings August 7, 2026 09:35

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

Suppressed comments (1)

CONTRIBUTING.md:357

  • In the label list, add a comma before “or other label” for correct punctuation; as written, it reads like kind/security or other is a single label.
For new features or larger work, assign the issue to the developer if
known, or add a `kind/release`, `kind/packages`, `kind/security` or other label
once the owning group is absolutely clear, and link it to the relevant milestone.

Copilot AI review requested due to automatic review settings August 7, 2026 09:39
Signed-off-by: Sayan Chowdhury <sayan.chowdhury2012@gmail.com>
Signed-off-by: Sayan Chowdhury <sayan.chowdhury2012@gmail.com>

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

Suppressed comments (2)

CONTRIBUTING.md:388

  • Spelling is inconsistent with the rest of the repository’s docs: elsewhere the project uses “behavior” (e.g., RELEASES.md), but this new section uses “behaviour”.
- Expected vs. actual behaviour

CONTRIBUTING.md:39

  • The new Table of Contents entries (AI-Assisted Contributions, Pull Request Etiquette, etc.) are indented as siblings of “Authoring PRs”, but their corresponding headings are ### sections inside the “Authoring PRs” section. This breaks the TOC hierarchy and makes these entries appear as top-level Development items rather than sub-items of Authoring PRs.
    - [AI-Assisted Contributions](#ai-assisted-contributions)
    - [Pull Request Etiquette](#pull-request-etiquette)
    - [Issue Assignment](#issue-assignment)
    - [Issue Quality Bar](#issue-quality-bar)
    - [Coordinated or Bulk-Filed Issues and PRs](#coordinated-or-bulk-filed-issues-and-prs)

Comment thread CONTRIBUTING.md
Signed-off-by: Sayan Chowdhury <sayan.chowdhury2012@gmail.com>
Copilot AI review requested due to automatic review settings August 7, 2026 10:40

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

Suppressed comments (1)

CONTRIBUTING.md:411

  • The table of contents links to a non-existent #reporting-spam-or-abuse section. The closest matching section below is titled Low-Quality Activity Is Discouraged, so the TOC entry currently becomes a broken link.
### Low-Quality Activity Is Discouraged

@dongsupark dongsupark left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks like a good start.

@sayanchowdhury sayanchowdhury changed the title Update the CONTRIBUTING.md to fight the recent spam Update the CONTRIBUTING.md to build quality in the recent PRs Aug 11, 2026
Copilot AI review requested due to automatic review settings September 1, 2026 16:14
@t-lo
t-lo merged commit fd8f215 into main Sep 1, 2026
2 checks passed
@t-lo
t-lo deleted the update-contributing.md branch September 1, 2026 16:15

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

Comment thread CONTRIBUTING.md
Comment on lines +405 to +411
### Low-Quality Activity Is Discouraged

Low-effort, AI-generated contributions have become common - bulk account
creation, near-identical low-effort issues or PRs, and scripted comment
pinging are patterns we see regularly, and this project strongly discourages
them. If you notice one, flag it to maintainers on Discord rather than
replying in the thread.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

7 participants