mirror of
https://github.com/containers/ansible-podman-collections.git
synced 2026-02-04 07:11:49 +00:00
Fix PR template (#956)
Signed-off-by: Sagi Shnaidman <sshnaidm@redhat.com>
This commit is contained in:
parent
2de46dd4de
commit
28126094cc
2 changed files with 46 additions and 86 deletions
46
.github/pull_request_template.md
vendored
Normal file
46
.github/pull_request_template.md
vendored
Normal file
|
|
@ -0,0 +1,46 @@
|
|||
# Type of change
|
||||
|
||||
Please select the type of change:
|
||||
|
||||
- [ ] Bug fix (non-breaking change which fixes an issue)
|
||||
- [ ] New feature (non-breaking change which adds functionality)
|
||||
- [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
|
||||
- [ ] Documentation update
|
||||
- [ ] CI/CD improvements
|
||||
- [ ] Refactoring
|
||||
- [ ] Other: _____
|
||||
|
||||
## Description
|
||||
|
||||
Brief description of the changes...
|
||||
|
||||
## Motivation and context
|
||||
|
||||
Why is this change required? What problem does it solve?
|
||||
|
||||
Fixes issue #___
|
||||
|
||||
## How has this been tested?
|
||||
|
||||
Describe the tests that you ran to verify your changes:
|
||||
|
||||
- [ ] Unit tests
|
||||
- [ ] Integration tests
|
||||
- [ ] Manual testing
|
||||
- [ ] Other: _____
|
||||
|
||||
## Checklist
|
||||
|
||||
- [ ] I have performed a self-review of my own code
|
||||
- [ ] I have tested my changes thoroughly
|
||||
- [ ] I have made corresponding changes to the documentation
|
||||
- [ ] I have added tests that prove my fix is effective or that my feature works
|
||||
- [ ] If I used AI tools, I have disclosed their use in the description of my changes and reviewed the output for accuracy
|
||||
|
||||
## Breaking changes
|
||||
|
||||
If this is a breaking change, describe what breaks and how to migrate:
|
||||
|
||||
## Additional notes
|
||||
|
||||
Any additional information or notes for reviewers:
|
||||
86
.github/pull_request_template.yml
vendored
86
.github/pull_request_template.yml
vendored
|
|
@ -1,86 +0,0 @@
|
|||
name: Pull Request
|
||||
description: Create a pull request
|
||||
body:
|
||||
- type: markdown
|
||||
attributes:
|
||||
value: |
|
||||
Thanks for contributing to the containers.podman collection!
|
||||
|
||||
- type: dropdown
|
||||
id: type
|
||||
attributes:
|
||||
label: Type of change
|
||||
description: What type of change does this PR introduce?
|
||||
options:
|
||||
- Bug fix (non-breaking change which fixes an issue)
|
||||
- New feature (non-breaking change which adds functionality)
|
||||
- Breaking change (fix or feature that would cause existing functionality to not work as expected)
|
||||
- Documentation update
|
||||
- CI/CD improvements
|
||||
- Refactoring
|
||||
- Other
|
||||
validations:
|
||||
required: true
|
||||
|
||||
- type: textarea
|
||||
id: description
|
||||
attributes:
|
||||
label: Description
|
||||
description: Describe what this PR does
|
||||
placeholder: Brief description of the changes...
|
||||
validations:
|
||||
required: true
|
||||
|
||||
- type: textarea
|
||||
id: motivation
|
||||
attributes:
|
||||
label: Motivation and context
|
||||
description: Why is this change required? What problem does it solve?
|
||||
placeholder: Fixes issue #123, adds support for...
|
||||
validations:
|
||||
required: false
|
||||
|
||||
- type: textarea
|
||||
id: testing
|
||||
attributes:
|
||||
label: How has this been tested?
|
||||
description: Describe the tests that you ran to verify your changes
|
||||
placeholder: |
|
||||
- [ ] Unit tests
|
||||
- [ ] Integration tests
|
||||
- [ ] Manual testing
|
||||
validations:
|
||||
required: false
|
||||
|
||||
- type: checkboxes
|
||||
id: checklist
|
||||
attributes:
|
||||
label: Checklist
|
||||
description: Please confirm the following
|
||||
options:
|
||||
- label: I have performed a self-review of my own code
|
||||
required: false
|
||||
- label: I have tested my changes thoroughly
|
||||
required: false
|
||||
- label: I have made corresponding changes to the documentation
|
||||
required: false
|
||||
- label: I have added tests that prove my fix is effective or that my feature works
|
||||
required: false
|
||||
- label: If I used AI tools, I have disclosed their use in the description of my changes and reviewed the output for accuracy
|
||||
required: false
|
||||
|
||||
- type: textarea
|
||||
id: breaking-changes
|
||||
attributes:
|
||||
label: Breaking changes
|
||||
description: If this is a breaking change, describe what breaks and how to migrate
|
||||
validations:
|
||||
required: false
|
||||
|
||||
- type: textarea
|
||||
id: additional
|
||||
attributes:
|
||||
label: Additional notes
|
||||
description: Any additional information or notes for reviewers
|
||||
validations:
|
||||
required: false
|
||||
Loading…
Add table
Add a link
Reference in a new issue