-
Notifications
You must be signed in to change notification settings - Fork 50
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Copies the latest template over from the ... template.
- Loading branch information
1 parent
3065ba8
commit c05f32d
Showing
1 changed file
with
28 additions
and
17 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,26 +1,37 @@ | ||
## Type of change | ||
## ๐๏ธ Tracking | ||
|
||
<!-- (mark with an `X`) --> | ||
<!-- Paste the link to the Jira or GitHub issue or otherwise describe / point to where this change is coming from. --> | ||
|
||
``` | ||
- [ ] Bug fix | ||
- [ ] New feature development | ||
- [ ] Tech debt (refactoring, code cleanup, dependency upgrades, etc) | ||
- [ ] Build/deploy pipeline (DevOps) | ||
- [ ] Other | ||
``` | ||
## ๐ Objective | ||
|
||
## Objective | ||
<!-- Describe what the purpose of this PR is, for example what bug you're fixing or new feature you're adding. --> | ||
|
||
<!--Describe what the purpose of this PR is. For example: what bug you're fixing or what new feature you're adding--> | ||
## ๐ธ Screenshots | ||
|
||
## Code changes | ||
<!-- Required for any UI changes; delete if not applicable. Use fixed width images for better display. --> | ||
|
||
<!--Explain the changes you've made to each file or major component. This should help the reviewer understand your changes--> | ||
<!--Also refer to any related changes or PRs in other repositories--> | ||
## โฐ Reminders before review | ||
|
||
- **file.ext:** Description of what was changed and why | ||
- Contributor guidelines followed | ||
- All formatters and local linters executed and passed | ||
- Written new unit and / or integration tests where applicable | ||
- Protected functional changes with optionality (feature flags) | ||
- Used internationalization (i18n) for all UI strings | ||
- CI builds passed | ||
- Communicated to DevOps any deployment requirements | ||
- Updated any necessary documentation (Confluence, contributing docs) or informed the documentation | ||
team | ||
|
||
## Before you submit | ||
## ๐ฆฎ Reviewer guidelines | ||
|
||
- Please add **unit tests** where it makes sense to do so | ||
<!-- Suggested interactions but feel free to use (or not) as you desire! --> | ||
|
||
- ๐ (`:+1:`) or similar for great changes | ||
- ๐ (`:memo:`) or โน๏ธ (`:information_source:`) for notes or general info | ||
- โ (`:question:`) for questions | ||
- ๐ค (`:thinking:`) or ๐ญ (`:thought_balloon:`) for more open inquiry that's not quite a confirmed | ||
issue and could potentially benefit from discussion | ||
- ๐จ (`:art:`) for suggestions / improvements | ||
- โ (`:x:`) or โ ๏ธ (`:warning:`) for more significant problems or concerns needing attention | ||
- ๐ฑ (`:seedling:`) or โป๏ธ (`:recycle:`) for future improvements or indications of technical debt | ||
- โ (`:pick:`) for minor or nitpick changes |