Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Proposed PR template #458

Merged
merged 2 commits into from
Aug 22, 2024
Merged

Proposed PR template #458

Changes from 1 commit
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
25cc9df
Proposed PR template
rocktavious Aug 22, 2024
109f8dd
Apply suggestions from code review
rocktavious Aug 22, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
24 changes: 20 additions & 4 deletions .github/pull_request_template.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,9 +1,25 @@
## Issues
Resolves #

<!-- paste an issue link here from github/gitlab -->
### Problem

## Changelog
<!---
Describe the problem this PR is solving. What is the application state
before this PR is merged?
-->

### Solution

<!---
Describe the way this PR solves the above problem. Add as much detail as you
can to help reviewers understand your changes. Include any alternatives and
tradeoffs you considered.
-->

### Checklist

- [ ] I have run this code, and it appears to resolve the stated issue.
- [ ] This PR does not reduce total test coverage
- [ ] This PR has no user interface changes or has already received approval from product management to change the interface.
- [ ] Does this change require a Terraform schema change?
- If so what is the ticket or PR #
- [ ] Make a `changie` entry
- [ ] Make a [`changie`](https://github.com/OpsLevel/opslevel-go/blob/main/CONTRIBUTING.md#changie-change-log-generation) entry that explans the customer facing outcome of this change
rocktavious marked this conversation as resolved.
Show resolved Hide resolved
rocktavious marked this conversation as resolved.
Show resolved Hide resolved