Skip to content

Commit

Permalink
Merge pull request #1078 from uktrade/remove-changelog-requirement-in-pr
Browse files Browse the repository at this point in the history
KLS-515 Remove changelog requirement in pull request
  • Loading branch information
hal274 authored Apr 11, 2023
2 parents 5495762 + bcb4d41 commit 1fc2195
Showing 1 changed file with 27 additions and 8 deletions.
35 changes: 27 additions & 8 deletions pull_request_template.md
Original file line number Diff line number Diff line change
@@ -1,8 +1,27 @@
To do (delete all that do not apply):

- [ ] Change has a jira ticket that has the correct status.
- [ ] Changelog entry added.
- [ ] (if there are vulnerable requirements) Upgraded any vulnerable dependencies.
- [ ] (if updating requirements) Requirements have been compiled.
- [ ] (if adding env vars) Added any new environment variable to vault.
- [ ] (if adding feature flags) Cleaned up old flags
(Please add a sentence or two explaining the context of this PR. Reviewers can go to the ticket for detail.)
CONTEXT: This changeset adds/removes/updates [summary of feature/area/code] so that [summary of benefit].

_Tick or delete as appropriate:_

### Workflow

- [ ] Ticket exists in Jira https://uktrade.atlassian.net/browse/TICKET_ID_HERE
- [ ] Jira ticket has the correct status.
- [ ] A clear/description pull request messaged added.

### Reviewing help

- [ ] Explains how to test locally, including how to set up appropriate data
- [ ] Includes screenshot(s) - ideally before and after, but at least after

### Housekeeping

- [ ] Added all new environment variables to Vault.
- [ ] Cleaned up old feature flags
- [ ] Upgraded any vulnerable dependencies.
- [ ] I have updated security dependencies
- [ ] Python requirements have been re-compiled.

### Merging

- [x] This PR can be merged by reviewers. (If unticked, please leave for the author to merge)

0 comments on commit 1fc2195

Please sign in to comment.