Skip to content

[no-ci] docs: clarify contributor coordination and issue readiness - #3103

Open
rwgk wants to merge 3 commits into
NVIDIA:mainfrom
rwgk:rwgk/contributor-review-coordination
Open

rwgk wants to merge 3 commits into
NVIDIA:mainfrom
rwgk:rwgk/contributor-review-coordination

Conversation

@rwgk

@rwgk rwgk commented Oct 14, 2026

Copy link
Copy Markdown
Contributor

Description

An open issue can capture an investigation or tentative proposal before an implementation plan has been agreed. Make that distinction clear so contributors can coordinate useful work and avoid implementing approaches that maintainers are still evaluating.

  • Clarify when contributors should coordinate scope and validation with maintainers, while welcoming bug reports, evidence, and direct PRs for small standalone corrections.
  • Explain responsibility for agent-assisted contributions: provide relevant context, understand the complete diff, keep changes focused, and report actual validation and remaining gaps.
  • Align AGENTS.md with this guidance. Require prominent readiness notices for investigations and proposals awaiting agreement, distinguish them from routine bug reports and approved work, and update the notices when the direction changes.

Related: #3074, which illustrates the need to distinguish an investigation from implementation-ready work. The cache investigation remains open.

Validation

  • pre-commit run --files AGENTS.md CONTRIBUTING.md — passed.
  • pre-commit run --all-files — passed on the merged head, including link checking.
  • git diff --check main...HEAD — passed.

Checklist

  • Validation appropriate to this documentation-only change is complete; no runtime tests were added.
  • The documentation is up to date with these changes.

@copy-pr-bot

copy-pr-bot Bot commented Oct 14, 2026

Copy link
Copy Markdown
Contributor

Auto-sync is disabled for draft pull requests in this repository. Workflows must be run manually.

Contributors can view more details about this message here.

@rwgk rwgk added this to the cuda.core next milestone Oct 14, 2026
@rwgk rwgk added documentation Improvements or additions to documentation cuda.bindings Everything related to the cuda.bindings module cuda.core Everything related to the cuda.core module cuda.pathfinder Everything related to the cuda.pathfinder module labels Oct 14, 2026
@rwgk rwgk self-assigned this Oct 14, 2026
@rwgk rwgk changed the title docs: clarify contributor coordination and issue readiness [no-ci] docs: clarify contributor coordination and issue readiness Oct 14, 2026
@rwgk
rwgk marked this pull request as ready for review October 14, 2026 06:11
@coderabbitai

coderabbitai Bot commented Oct 14, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: Repository: NVIDIA/cuda-python/.coderabbit.yaml
  • Review profile: CHILL
  • Plan: Enterprise
  • Run ID: 934b6583-ee80-41e1-95a8-f57a45528b90
📥 Commits

Reviewing files that changed from the base of the PR and between c6a9bc5 and a7c2f6f.

📒 Files selected for processing: 1
  • CONTRIBUTING.md

Included review availability: This review used your included allowance. Your plan provides up to 12 included reviews per hour; 10 remain after this review.

📝 Summary

Summary by CodeRabbit

  • Documentation
    • Expanded contributor guidance for reporting problems, distinguishing issue evidence from proposed solutions, and coordinating implementation.
    • Clarified when to wait for assignment or maintainer agreement, and when small standalone fixes or documentation corrections can be proposed directly.
    • Added expectations for focused changes, validation, review, and coordination on shared infrastructure.
    • Added guidance for agent-assisted contributions, readiness notices, approved work, implementation changes, and protecting private information.

AGENTS.md and CONTRIBUTING.md add guidance for issue reporting, contribution coordination, issue readiness notices, shared infrastructure changes, and agent-assisted contributions.

Changes: Contribution Guidance

Layer / File(s) Summary
Scope and reporting
AGENTS.md, CONTRIBUTING.md
Adds issue-reporting details and guidance on authorized scope, maintainer coordination, and focused contributions.
Issue readiness and infrastructure
AGENTS.md, CONTRIBUTING.md
Adds guidance for distinguishing observed problems from proposals, setting and updating readiness notices, and coordinating changes to shared infrastructure.
Agent-assisted contributions
CONTRIBUTING.md
Adds expectations for providing context, understanding the full diff, reporting validation and gaps, following test-authorship markers, and staying within agreed scope.

Suggested reviewers: mdboom

Priority: ⬇️ Low

Change: Other

Merge Risk: ⚪ Minimal · up to a7c2f

This change clarifies contributor expectations without introducing a concrete workflow conflict. It is ready to merge after normal checks.

- [ ] Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Comment @coderabbitai help to get the list of available commands.

@rwgk rwgk added the PR review get-together Mark PRs you'd like the team to review at the weekly PR review get-together. label Oct 17, 2026

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cuda.bindings Everything related to the cuda.bindings module cuda.core Everything related to the cuda.core module cuda.pathfinder Everything related to the cuda.pathfinder module documentation Improvements or additions to documentation PR review get-together Mark PRs you'd like the team to review at the weekly PR review get-together.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant