From c05f32d3edcc99aa2e2ab4c0c92d2c3aa1bc09c9 Mon Sep 17 00:00:00 2001 From: Matt Bishop Date: Fri, 24 May 2024 11:33:23 -0400 Subject: [PATCH] Use latest PR template (#799) Copies the latest template over from the ... template. --- .github/PULL_REQUEST_TEMPLATE.md | 45 ++++++++++++++++++++------------ 1 file changed, 28 insertions(+), 17 deletions(-) diff --git a/.github/PULL_REQUEST_TEMPLATE.md b/.github/PULL_REQUEST_TEMPLATE.md index de0ba909c..a5e6b294d 100644 --- a/.github/PULL_REQUEST_TEMPLATE.md +++ b/.github/PULL_REQUEST_TEMPLATE.md @@ -1,26 +1,37 @@ -## Type of change +## 🎟ī¸ Tracking - + -``` -- [ ] Bug fix -- [ ] New feature development -- [ ] Tech debt (refactoring, code cleanup, dependency upgrades, etc) -- [ ] Build/deploy pipeline (DevOps) -- [ ] Other -``` +## 📔 Objective -## Objective + - +## 📸 Screenshots -## Code changes + - - +## ⏰ 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 + + +- 👍 (`:+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