forked from IPPETAD/adventure.datetime
-
Notifications
You must be signed in to change notification settings - Fork 0
Coding Standards
Andrew Fontaine edited this page Oct 16, 2013
·
4 revisions
##Pull Requests##
Pull requests require approval of two other team members. First team member should comment their approval (or any issues) on the request. Second member can then allow the merge if the first has approved.
Pull requests should mention the issue they are addressing. Github makes this easy by adding the issue number in the commit message and pull request title with a '#' preceding. (See #10 for example).
##Forks##
Everyone has their own fork of the repo. If you need someone's changes before they're merged into the main repo, just git fetch $NAME and git merge $NAME/$BRANCH to continue working.
To get the remote repos set up right, follow the handy repository set up guide here.