alex/valence
1
0
Fork 0
mirror of https://github.com/italicsjenga/valence.git synced 2024-12-23 14:31:30 +11:00
Code Issues Packages Projects Releases Wiki Activity

Remove "test plan" from pull request template (#336)

Browse source 
## 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).
This commit is contained in:
Ryan Johnson 2023-05-23 05:45:15 -07:00 committed by GitHub
parent 6d1a29daa4
commit ad1f5d1b4c
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 0 additions and 17 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

17
.github/pull_request_template.md vendored
View file

@ -1,20 +1,3 @@
## Description ## Description
Describe the changes you've made. Link to any issues this PR fixes or addresses. 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>