-
Notifications
You must be signed in to change notification settings - Fork 85
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Update PR template from template repo (#574)
- Loading branch information
Showing
1 changed file
with
24 additions
and
23 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,33 +1,34 @@ | ||
## Type of change | ||
## 🎟️ Tracking | ||
|
||
- [ ] Bug fix | ||
- [ ] New feature development | ||
- [ ] Tech debt (refactoring, code cleanup, dependency upgrades, etc) | ||
- [ ] Build/deploy pipeline (DevOps) | ||
- [ ] Other | ||
<!-- Paste the link to the Jira or GitHub issue or otherwise describe / point to where this change is coming from. --> | ||
|
||
## Objective | ||
## 📔 Objective | ||
|
||
<!--Describe what the purpose of this PR is. For example: what bug you're fixing or what new feature you're adding--> | ||
<!-- Describe what the purpose of this PR is, for example what bug you're fixing or new feature you're adding. --> | ||
|
||
## Code changes | ||
## 📸 Screenshots | ||
|
||
<!--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--> | ||
<!-- Required for any UI changes; delete if not applicable. Use fixed width images for better display. --> | ||
|
||
- **file.ext:** Description of what was changed and why | ||
## ⏰ Reminders before review | ||
|
||
## Screenshots | ||
- Contributor guidelines followed | ||
- All formatters and local linters executed and passed | ||
- Written new unit and / or integration tests where applicable | ||
- 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 | ||
|
||
<!--Required for any UI changes. Delete if not applicable--> | ||
## 🦮 Reviewer guidelines | ||
|
||
## Testing requirements | ||
<!-- Suggested interactions but feel free to use (or not) as you desire! --> | ||
|
||
<!--What functionality requires testing by QA? This includes testing new behavior and regression testing--> | ||
|
||
## Before you submit | ||
|
||
- [ ] I have checked for **linting** errors (`npm run lint`) (required) | ||
- [ ] I have added **unit tests** where it makes sense to do so (encouraged but not required) | ||
- [ ] This change requires a **documentation update** (notify the documentation team) | ||
- [ ] This change has particular **deployment requirements** (notify the DevOps team) | ||
- 👍 (`:+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 |