From 30c8dabeb4d2f0683ba372a315fd35a7b302c216 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jakub=20=C5=BD=C3=A1dn=C3=ADk?= Date: Thu, 16 Jun 2022 16:14:59 +0300 Subject: [PATCH] Add test requirements to PR template (#5809) * Add test requirements to PR template * tweak words * another tweak * add /tests folder as a suggestion Co-authored-by: Darren Schroeder <343840+fdncred@users.noreply.github.com> --- .github/pull_request_template.md | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/.github/pull_request_template.md b/.github/pull_request_template.md index 75f0b3335c..0162a14674 100644 --- a/.github/pull_request_template.md +++ b/.github/pull_request_template.md @@ -4,6 +4,12 @@ # Tests +Make sure you've done the following: + +- [ ] Add tests that cover your changes, either in the command examples, the crate/tests folder, or in the /tests folder. +- [ ] Try to think about corner cases and various ways how your changes could break. Cover them with tests. +- [ ] If adding tests is not possible, please document in the PR body a minimal example with steps on how to reproduce so one can verify your change works. + Make sure you've run and fixed any issues with these commands: - [ ] `cargo fmt --all -- --check` to check standard code formatting (`cargo fmt --all` applies these changes)