Repository navigation
docs: add AI contribution governance - #1801
michael-valdron wants to merge 1 commit into
Conversation
Document requirements for AI-assisted contributions and link them from contributor guidance and the pull request template. Assisted-by: GPT-5 <noreply@openai.com> Signed-off-by: Michael Valdron <mvaldron@redhat.com>
|
No actionable comments were generated in the recent review.
 |
| Layer / File(s) | Summary |
|---|---|
AI contribution governance requirements GOVERNANCE.md |
The governance document adds standards for AI-assisted contributions, contributor checks for tool terms and third-party permissions, attribution requirements, and a link to the Linux Foundation policy. |
Contribution and pull request guidance CONTRIBUTING.md, .github/PULL_REQUEST_TEMPLATE.md |
The contribution guide requires AI-assisted contributions to follow the governance requirements. The pull request template adds an optional checkbox that links to those requirements. |
Priority:
Estimated code review effort: 2 (Simple) | ~5 minutes
Change: Other
Merge Risk: 
03930
-
This documentation-only change is ready to merge with no identified functional or operational risk.
Pre-merge checks: 
-
Check name Status Explanation Description Check Check skipped - CodeRabbit’s high-level summary is enabled. Title check The title clearly and concisely describes the main change: adding AI contribution governance documentation. Docstring Coverage No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentat… Linked Issues check Check skipped because no linked issues were found for this pull request. Out of Scope Changes check Check skipped because no linked issues were found for this pull request. Full details: Docstring Coverage
-
Explanation
No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 0 functions across 0 files. (3 skipped: 3 unsupported.)
-
Finishing Touches
-
🧪 Generate unit tests (beta)
-
- Create a new PR
-
- Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.
Comment @coderabbitai help to get the list of available commands.
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: JslYoon, michael-valdron The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
Description of Changes
Document requirements for AI-assisted contributions and link them from contributor guidance and the pull request template.
Related Issue(s)
#1773
Acceptance Criteria
Testing and documentation do not need to be complete in order for this PR to be approved. However, tracking issues must be opened for missing testing/documentation.
New testing and documentation issues can be opened under
devfile/api/issues.You can check the respective criteria below if either of the following is true:
If criteria is left unchecked please provide an explanation why.
Unit/Functional tests
QE Integration test
Documentation
Client Impact
Tests Performed
Explain what tests you personally ran to ensure the changes are functioning as expected.
How To Test
Instructions for the reviewer on how to test your changes.
Notes To Reviewer
Any notes you would like to include for the reviewer.
Summary by CodeRabbit