From 1cbaa43db520b6c256cadbd0666aa5653a1b9b13 Mon Sep 17 00:00:00 2001 From: Tim Abbott Date: Mon, 25 Nov 2024 10:28:12 -0800 Subject: [PATCH] github: Add a zulip/zulip style pull request template. Hopefully, this will help improve the quality of new-contirbutor pull requests to this project. --- .github/pull_request_template.md | 43 ++++++++++++++++++++++++++++++++ 1 file changed, 43 insertions(+) create mode 100644 .github/pull_request_template.md diff --git a/.github/pull_request_template.md b/.github/pull_request_template.md new file mode 100644 index 00000000..31d17ed5 --- /dev/null +++ b/.github/pull_request_template.md @@ -0,0 +1,43 @@ + + +Fixes: + + + +**Screenshots and how this was tested:** + +
+Self-review checklist + + + + + +- [ ] [Self-reviewed](https://zulip.readthedocs.io/en/latest/contributing/code-reviewing.html#how-to-review-code) the changes for clarity and maintainability + (variable names, code reuse, readability, etc.). + +Communicate decisions, questions, and potential concerns. + +- [ ] Explains differences from previous plans (e.g., issue description). +- [ ] Highlights technical choices and bugs encountered. +- [ ] Calls out remaining decisions and concerns. +- [ ] Automated tests verify logic where appropriate. + +Individual commits are ready for review (see [commit discipline](https://zulip.readthedocs.io/en/latest/contributing/commit-discipline.html)). + +- [ ] Each commit is a coherent idea. +- [ ] Commit message(s) explain reasoning and motivation for changes. + +Completed manual review and testing of the following: + +- [ ] Visual appearance of the changes. +- [ ] Responsiveness and internationalization. +- [ ] Strings and tooltips. +- [ ] End-to-end functionality of buttons, interactions and flows. +- [ ] Corner cases, error conditions, and easily imagined bugs. +