From ad1f5d1b4c9be45f2abb69d7a3ef9d30476fdcd6 Mon Sep 17 00:00:00 2001 From: Ryan Johnson <ryanj00a@gmail.com> Date: Tue, 23 May 2023 05:45:15 -0700 Subject: [PATCH] Remove "test plan" from pull request template (#336) ## Description Remove the "test plan" section from the pull request template. My justification is: - The steps needed to test a pull request is usually obvious. - Often the test plan is just "run `cargo test` lol" - For small changes and tweaks, a "test plan" is overkill and amounts to frustrating busywork. - Adds too much additional friction for new contributors. - Large projects like Bevy are fine without it. - Often not applicable to pull requests (like this one). --- .github/pull_request_template.md | 17 ----------------- 1 file changed, 17 deletions(-) diff --git a/.github/pull_request_template.md b/.github/pull_request_template.md index f6c18d6..4917846 100644 --- a/.github/pull_request_template.md +++ b/.github/pull_request_template.md @@ -1,20 +1,3 @@ ## Description Describe the changes you've made. Link to any issues this PR fixes or addresses. - -## Test Plan - -Explain the steps necessary to test your changes. If you used a playground, include the code in the details below. - -Steps: -1. - -<details> - -<summary>Playground</summary> - -```rust -PASTE YOUR PLAYGROUND CODE HERE -``` - -</details>