Skip to content

Commit 48da715

Browse files
committed
docs(contributing): add contribution intake and decision record
Signed-off-by: Will Curran <wcurran@nvidia.com>
1 parent 01e0b92 commit 48da715

2 files changed

Lines changed: 85 additions & 3 deletions

File tree

.github/ISSUE_TEMPLATE/feature_request.yml

Lines changed: 70 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -21,13 +21,80 @@ body:
2121
validations:
2222
required: true
2323

24+
- type: markdown
25+
attributes:
26+
value: |
27+
## Proposal Details
28+
29+
Answer each required proposal-detail field.
30+
If a field does not apply, enter `Not applicable` and explain why.
31+
If the answer is unknown, enter `Unknown` and state what must be decided.
32+
2433
- type: textarea
2534
id: constraints
2635
attributes:
27-
label: Constraints and Non-goals
28-
description: List only current constraints or exclusions that materially change the solution.
36+
label: Scope and Exclusions
37+
description: |
38+
State what the proposal includes and excludes.
39+
List current constraints that materially change the solution.
2940
validations:
30-
required: false
41+
required: true
42+
43+
- type: textarea
44+
id: ownership
45+
attributes:
46+
label: Ongoing Ownership
47+
description: |
48+
Who will maintain this contribution after merge?
49+
Name the proposed person or team.
50+
validations:
51+
required: true
52+
53+
- type: textarea
54+
id: placement_support
55+
attributes:
56+
label: Placement and Support Expectations
57+
description: |
58+
Request placement in core NemoClaw or NemoClaw Community.
59+
State the support expected after merge.
60+
Maintainers make the final placement and support decision.
61+
Refer to [Community Solutions](https://github.com/NVIDIA/NemoClaw/blob/main/docs/resources/community-contributions.mdx).
62+
validations:
63+
required: true
64+
65+
- type: textarea
66+
id: validation_plan
67+
attributes:
68+
label: Validation Plan
69+
description: |
70+
List the tests, documentation, security analysis, compatibility evidence,
71+
or runtime validation required to evaluate this proposal.
72+
validations:
73+
required: true
74+
75+
- type: textarea
76+
id: compatibility
77+
attributes:
78+
label: Compatibility Requirements
79+
description: |
80+
List the versions, operating systems, hardware, providers, and runtimes
81+
that this proposal is expected to work with. State any exclusions.
82+
validations:
83+
required: true
84+
85+
- type: textarea
86+
id: security_privacy
87+
attributes:
88+
label: Security or Privacy Impact
89+
description: |
90+
Describe changes to permissions, credentials, personal data, network access,
91+
execution boundaries, or third-party trust.
92+
Do not include credential values, personal data, or confidential customer
93+
or partner information.
94+
Do not report a vulnerability in this issue.
95+
Follow [SECURITY.md](https://github.com/NVIDIA/NemoClaw/blob/main/SECURITY.md).
96+
validations:
97+
required: true
3198

3299
- type: textarea
33100
id: implementation

CONTRIBUTING.md

Lines changed: 15 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -79,6 +79,21 @@ Open an issue when you encounter one of the following situations.
7979
Use [GitHub Discussions](https://github.com/NVIDIA/NemoClaw/discussions) for questions, design exploration, and larger feature proposals before implementation.
8080
Maintainers may ask you to move broad or still-forming proposals from an issue to a discussion so the design can settle before code review.
8181

82+
### Maintainer Triage and Decision
83+
84+
For a substantive product, architecture, security, integration, or supported-surface proposal, a maintainer must record a decision in the issue or linked discussion.
85+
The maintainer must record the decision before implementation starts.
86+
87+
The decision record must include these items:
88+
89+
- **Decision:** `Accept`, `Request changes`, `Defer`, or `Decline`.
90+
- **Reason and placement:** State the reason. For `Accept`, record core NemoClaw or NemoClaw Community. Use `Not applicable` when the decision is not `Accept`.
91+
- **Accountable maintainer:** Name one maintainer when the decision is `Accept`. Use `Not applicable` when the decision is not `Accept`.
92+
- **Validation plan:** List the required validation evidence when the decision is `Accept`. Use `Not applicable` when the decision is not `Accept`.
93+
94+
Contributors must not implement a substantive contribution with an `Accept` decision until the record names one accountable maintainer and an explicit validation plan.
95+
Small documentation changes and low-risk fixes may continue directly to a pull request.
96+
8297
## Community Response Expectations
8398

8499
NemoClaw is an alpha project, and maintainer availability varies with release, security, and stability work.

0 commit comments

Comments
 (0)