From 04aa97f0fd67b2e4298eb8c950d2e62c3454baa6 Mon Sep 17 00:00:00 2001 From: psychedelicious <4822129+psychedelicious@users.noreply.github.com> Date: Thu, 14 Mar 2024 17:18:54 +1100 Subject: [PATCH] gh: update pr template --- .github/pull_request_template.md | 67 +++++++------------------------- 1 file changed, 13 insertions(+), 54 deletions(-) diff --git a/.github/pull_request_template.md b/.github/pull_request_template.md index 0f0b953be7..3f46dd6161 100644 --- a/.github/pull_request_template.md +++ b/.github/pull_request_template.md @@ -1,66 +1,25 @@ -## What type of PR is this? (check all applicable) + -- [ ] Refactor -- [ ] Feature -- [ ] Bug Fix -- [ ] Optimization -- [ ] Documentation Update -- [ ] Community Node Submission +## Summary + -## Have you discussed this change with the InvokeAI team? -- [ ] Yes -- [ ] No, because: +## Related Issues / Discussions - -## Have you updated all relevant documentation? -- [ ] Yes -- [ ] No + +## QA Instructions -## Description - - -## Related Tickets & Documents - - - -- Related Issue # -- Closes # - -## QA Instructions, Screenshots, Recordings - - + ## Merge Plan - -Example merge plans: -- "This PR can be merged when approved" -- "This must be squash-merged when approved" -- "DO NOT MERGE - I will rebase and tidy commits before merging" -- "#dev-chat on discord needs to be advised of this change when it is merged" +## Checklist -A merge plan is particularly important for large PRs or PRs that touch the -database in any way. ---> + -## Added/updated tests? - -- [ ] Yes -- [ ] No : _please replace this line with details on why tests - have not been included_ - -## [optional] Are there any post deployment tasks we need to perform? +- [ ] The PR has a short but descriptive title +- [ ] Tests added / updated +- [ ] Documentation added / updated