-
Notifications
You must be signed in to change notification settings - Fork 2
Git Style
Rafi Long edited this page May 10, 2014
·
3 revisions
- Pushes
- Content (YAML) -> master
- Functionality (HTML, CSS, JS) -> dev
- Broken (committed) -> local only
- Branches
- dev
- ready to be merged with master at all times
- master
- only content (YAML)
- new branches
- anything broken
- Requirements
- Creation requirements
- One of...
- Multiple people
- Long term
- Agreed upon
- Must be deleted after work is finished
- Will be merged with dev
- Steps
- Pull origin/dev to dev
- Create new branch
- Work on new branch
- Commit
- Switch to dev
- Pull origin/dev to dev
- Merge new branch to dev
- Push
- Delete accordian
- One or more commits per feature
- Descriptive
- Terse
- No names
- Minimal swear words
- Merge commits
- Start with merge
- Follow APA style
- Reference issues
- Use
resolves [issue number]
- Creation labels go through Max
- Label sparingly
- Must have a point
- Tags used sparingly & important polls
- Close via commit
- No jokes about issues
- Meta
- Only Max and Chris can make meta issues