Skip to content

Commit 8ecc5f4

Browse files
Update pull_request_template.md
Additional text regarding pull requests being reviewed by two developers. Little more information about test results. Added another reason for code rejection.
1 parent 3c55a7f commit 8ecc5f4

File tree

1 file changed

+7
-2
lines changed

1 file changed

+7
-2
lines changed

pull_request_template.md

+7-2
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,11 @@
1+
**Please read, especially if your first time contributing to PhantomBot:**
2+
13
Thank you for contributing to PhantomBot! Hopefully you followed the code style guide:
24
https://github.com/PhantomBot/PhantomBot/blob/master/CODESTYLE.md
35

46
If you added in code from a third party, it must have a license that is compatible with PhantomBot. If it is not, we will reject the merge request. The development team takes this very seriously. If you add in access to an API and PhantomBot would use that API improperly, we will reject the merge request. Again, the development team takes this very seriously.
57

6-
You must provide test results for your change. If test results are not provided we will do one of two things. We will ask you for test results or we will just reject the change.
8+
You must provide test results for your change. If test results are not provided we will do one of two things. We will ask you for test results or we will just reject the change. Test results convince us that you performed a unit test and verified your change.
79

810
We reserve the right to reject a change for any reason at all. Typically, we will provide a reason but if we do not, that is at our discretion. There are some reasons for which we will reject changes, other than the ones given above (but they will be repeated):
911

@@ -15,7 +17,10 @@ We reserve the right to reject a change for any reason at all. Typically, we wi
1517
- Items with potential to spam or consume the outgoing message queue.
1618
- Poorly architected items.
1719
- Item is supported in chat but no update for the Control Panel.
20+
- Any command that would violate the Twitch Terms of Service or the Terms of Service of any provider.
1821

1922
You are not to harass the development team if a pull request is rejected. You may discuss and argue why you disagree with a rejection. We do ask that you stay professional and amicable in your argument and discussion.
2023

21-
**You may delete this note from your Pull Request if you so desire. It is acceptable if you leave it in place.**
24+
All pull requests will be reviewed by at least two PhantomBot developers. Please be patient while two developers take the time to do so. We understand that this adds time to an approval, but it is to ensure that the development team is not missing anything.
25+
26+
**You may delete this entire text from your Pull Request if you so desire. It is acceptable if you leave it in place as well.**

0 commit comments

Comments
 (0)