Repository navigation
Update PR Contributor Checklist #8244
Description
Area for Improvement
PR contributor checklist: https://github.com/radius-project/radius/blob/main/.github/pull_request_template.md#contributor-checklist
Observed behavior
The current checklist doesn't provide a clear checkbox for scenarios where individual items are not applicable.
As an example, we have a checkbox for "An overview of proposed schema changes is included in a linked GitHub issue.".. but it doesn't guide the author on what to do if this is not applicable for the corresponding PR contribution - check the box/leave it empty/something else?
Desired behavior
Make it explicit for scenarios where the checklist item doesn't apply to the PR.
Proposed Fix
Add two checkbox options on each of the items - "Yes" and "Not Applicable". Based on the PR, one of the two options should be checked.
rad Version
N/A
Operating system
No response
Additional context
No response
Would you like to support us?
- Yes, I would like to support you
Activity
radius-triage commentedon Jan 15, 2025
👋 @kachawla Thanks for filing this issue.
A project maintainer will review this issue and get back to you soon.
We also welcome community contributions! If you would like to pick this item up sooner and submit a pull request, please visit our contribution guidelines and assign this to yourself by commenting "/assign" on this issue.
For more information on our triage process please visit our triage overview
radius-triage commentedon Jan 15, 2025
We've prioritized work on this issue. Please subscribe to this issue for notifications, we'll provide updates as we make progress.
We also welcome community contributions! If you would like to pick this item up sooner and submit a pull request, please visit our contribution guidelines and assign this to yourself by commenting "/assign" on this issue.
For more information on our triage process please visit our triage overview
kachawla commentedon Jan 15, 2025
I'm going to take care of it, this week or next.
Update PR Contributor Checklist (#8255)