Skip to content

Commit

Permalink
Merge pull request #1650 from pllim/update-changelog-policy
Browse files Browse the repository at this point in the history
DEV: Clarify change log policy
  • Loading branch information
pllim authored Sep 15, 2022
2 parents d351f25 + fbb98ef commit f1849e8
Showing 1 changed file with 5 additions and 1 deletion.
6 changes: 5 additions & 1 deletion .github/PULL_REQUEST_TEMPLATE.md
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,11 @@ the following line. -->

Fixes #<Issue Number>

### Change log entry

- [ ] Is a change log needed? If yes, is it added to `CHANGES.rst`? If you want to avoid merge conflicts,
list the proposed change log here for review and add to `CHANGES.rst` before merge.

### Checklist for package maintainer(s)
<!-- This section is to be filled by package maintainer(s) who will
review this pull request. -->
Expand All @@ -32,6 +37,5 @@ This checklist is meant to remind the package maintainer(s) who will review this
- [ ] Are tests added/updated as required? If so, do they follow the [STScI Style Guides](https://github.com/spacetelescope/style-guides)?
- [ ] Are docs added/updated as required? If so, do they follow the [STScI Style Guides](https://github.com/spacetelescope/style-guides)?
- [ ] Did the CI pass? If not, are the failures related?
- [ ] Is a change log needed? If yes, is it added to `CHANGES.rst`?
- [ ] Is a milestone set?
- [ ] After merge, any internal documentations need updating (e.g., JIRA, Innerspace)?

0 comments on commit f1849e8

Please sign in to comment.