andre/30-seconds-of-code
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Updated for transfer to org

Browse Source
This commit is contained in:
Angelos Chalaris
2018-09-22 10:58:26 +03:00
parent 8dbc7be7ad
commit cf8acbcb1f
21 changed files with 33 additions and 33 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
COLLABORATING.md
View File

@ -13,7 +13,7 @@ As a member of the team that manages **30 seconds of code**, you have the follow
## Guidelines for merging pull requests and making changes to the project
- **[Usual guidelines](https://github.com/Chalarangelo/30-seconds-of-code/blob/master/CONTRIBUTING.md) apply.** Make sure to follow them, like everybody else.
- **[Usual guidelines](https://github.com/30-seconds/30-seconds-of-code/blob/master/CONTRIBUTING.md) apply.** Make sure to follow them, like everybody else.
- **For a pull request to be considered ready to merge, there should be at least 2 (preferably 3) reviews approving it for merge.** There are, however, certain exceptions:
- **If a pull request only fixes typos**, there is no need to wait for a second reviewer (unless you are not certain these were not typos in the first place).
- **If a pull request only clarifies a snippet's description or enforces the style guide for an existing snippet**, you might be able to merge it without getting a second reviewer to review it, but only if you are certain about it.