diff options
author | Skjalf <47818697+Nyeriah@users.noreply.github.com> | 2023-08-10 08:45:33 -0300 |
---|---|---|
committer | GitHub <noreply@github.com> | 2023-08-10 08:45:33 -0300 |
commit | 4f20e277945c4c1738ee1491d23da6f5faf24eb0 (patch) | |
tree | 925a10ca02324f013c5440751abc31d377e960dd | |
parent | d6dfcf2dcfc72ecc1261826645cb79a1ef5570d7 (diff) |
docs: Simplify the pull request template (#16869)
* docs: Simplify the pull request template
* fix typo
* Update pull_request_template.md
* Update pull_request_template.md
* Update pull_request_template.md
-rw-r--r-- | pull_request_template.md | 22 |
1 files changed, 18 insertions, 4 deletions
diff --git a/pull_request_template.md b/pull_request_template.md index 3b677b0d2b..7f6c343dcc 100644 --- a/pull_request_template.md +++ b/pull_request_template.md @@ -1,8 +1,12 @@ <!-- First of all, THANK YOU for your contribution. --> ## Changes Proposed: -- -- +This PR proposes changes to: +- [ ] Core (units, players, creatures, game systems). +- [ ] Scripts (bosses, spell scripts, creature scripts). +- [ ] Database (SAI, creatures, etc). + +If your pull request promotes complex changes that require a detailed explanation, please describe them in detail specifying what your solution is and what is it meant to address. ## Issues Addressed: <!-- If your fix has a relating issue, link it below --> @@ -10,16 +14,26 @@ ## SOURCE: <!-- If you can, include a source that can strengthen your claim --> +The changes have been validated through: +- [ ] Live research (checked on live servers, e.g Classic WotLK, Retail, etc.) +- [ ] Sniffs (remember to share them with the open source community!) +- [ ] Video evidence, knowledge databases or other public sources (e.g forums, Wowhead, etc.) +- [ ] The changes promoted by this pull request come partially or entirely from another project (cherry-pick). **Cherry-picks must be committed using the proper --author tag in order to be accepted, thus crediting the original authors, unless otherwise unable to be found** ## Tests Performed: <!-- Does it build without errors? Did you test in-game? What did you test? On which OS did you test? Describe any other tests performed --> -- -- +This PR has been: +- [ ] Tested in-game by the author. +- [ ] Tested in-game by other community members/someone else other than the author/has been live on production servers. +- [ ] This pull request requires further testing and may have edge cases to be tested. ## How to Test the Changes: <!-- Describe in a detailed step-by-step order how to test the changes --> +- [ ] This pull request can be tested by following the reproduction steps provided in the linked issue +- [ ] This pull request requires further testing. Provide steps to test your changes. If it requires any specific setup e.g multiple players please specify it as well. + 1. 2. 3. |