Feb 13, 2014

Git Workflow

Document version: 0.5
Last update: 2014-03-03
Author: Anar Manafov (Anar.Manafov@gmail.com)

Introduction

This document describes a git workflow used by the development team. The workflow aims to provide:
  • uninterruptible development,
  • a stable and releasable at any time master,
  • a multilevel protection against conflicts,
  • a multilevel possibility to recover from errors/mistakes before changes comes into the master,
  • a clean history of the master branch without merge commits and other unwanted garbage.

Treat public history as immutable, atomic, and easy to follow. Treat private history as disposable and malleable.

Requirements and Tips

  • Use one branch per feature/bug (contained development). 
    Every tick/task/feature MUST be implemented on a separate branch. Basically each JIRA ticket should be represented by at least one branch.
  • Only release managers are allowed to work on the central master branch.
  • Cherry picking MUST not be used by any means.
  • A good use of branches should prevent the need of "git cherry-pick".
  • DO NOT create very large repositories.
  • DO NOT commit large binary files.
  • DO NOT commit any file, which can be regenerated or which is generated automatically by your development environment.
  • Remember to rebase your feature branch before merging it to the Development branch.
  • Specify the origin and branch when pushing (might avoid mistakes).
  • Use "git pull --rebase" in order to avoid merges from upstream commits.

The workflow


Important

In your development process, you must strictly distinguish between Features, Fixes and Hot Fixes.
  • FeaturesFixes - are patches, which can wait until the next release of the product. This is why they travail starting from private feature branches ---> the central dev branch ---> the central release branch ---> the central master
  • Hot Fixes - are the patches, which can't wait and must be applied directly on a release version of the product. This is why they have completely different path: a new private branch from the given tag on the central master ---> central master.

Branches

There are only a few long term branches - master and dev. All other branches should be deleted as soon as their commits are merged into the master or you don't need them.
Use the following command to find out branches, which a merged into the mainrepo/master:
git branch --merged mainrepo/master 
or not merged:
git branch --no-merged mainrepo/master 

master branch

Contains all the stable, released code.
  • All released versions of all modules should be tagged in the master.
  • No separate branches for the released versions.
  • The master branch is ready to build at any moment.
  • No development should be performed on the master branch directly.
  • Only release managers have write permissions on it.
  • The master branch rolls only forward, no history changes are allowed on the master branch.
  • All new patches are introduced in the master branch only via "git merge --ff-only".

dev branch

  • The branch is inherited from the latest master.
  • The dev branch is a development mainstream.
  • A release manager defines a list of tasks for a development sprint. For each task, developers MUST create a separate branch inherited from the dev branch.
  • Should be rebased from the master each time master is changed.

RC branch

RC or ReleaseCandidate branch is a temporary branch. This branch intends to provide uninterruptible development - to avoid such mails like "Please do not commit..., we freeze a branch" :).
At the moment of a release, a release manager will branch from dev to create an RC branch to start a release procedure. It also means feature freeze on the RC branch and only bug fixes can be introduced in it - no commits with new features. The fixes introduced in RC should be then synchronized into dev as well, to keep development mainstream up to date.
While a release manager works on the RC branch, all developers continue developing in dev and in their feature branches - uninterruptible development.
The branch should be deleted as soon as it has benn merged with the master.

HotFix branch

It is a branch for hot fixes. It should contain only urgent hot fixes for the released versions, which can't wait until the next scheduled release of the product.
Once a hot fix is merged back in to the master, the dev branch should be rebased from the master to get new changes.
The branch should be deleted as soon as merged with the master.

FEATURE/TICKET branches

Each business or development task/ticket/feature should have at least one feature branch. Such branches must be rebased to dev as often as possible to simplify future merging.
Rebasing is important to keep new commits together for possible tuning or squashing in the feature.
Whenever a feature is ready (or a stable part of the feature) it can be merged with the dev branch.
It is recommmended to keep feature branches even after their merge with dev. It will simplify fine tunning in case if the feature represented by the branch will be revereted from the dev for additional development or fixes/corrections.
Feature branches should be deleted as soon as their commits are merged into the master via the dev branch.

Roles

Release manager (r/w: MASTER, DEV, RC) 
Developer (r/w: FEATURE(s), HOT FIX; read only: MASTER, DEV)

User stories

In the following documentation we use:
  • mainrepo_url - the url of the main repo, which is restricted and managed by release managers only. Developers must not have write permitions on it.
  • mainrepo - the name of the main repo.
  • url_of_the_fork - the url of your fork repo of the main repo.
  • developerrepo_url - the url of a developer's repo.

Developers

Prepare the environment

  1. setup git configuration. The following you have yo execute on all of your machines, or copy the git config to all of your machines.
     git config --global branch.autosetuprebase always
     git config --global user.name "FirsName LastName"
     git config --global user.email johndoe@example.com
    
  2. Using github fork the main repo.
  3. Create a local copy of the forked repo:
     git clone url_of_the_fork
    
  4. Now when your origin points to your fork. You need to add the main repo to your remotes as well. You should have "origin --> you fork" and "mainrepo --> the main repo".
     git remote add mainrepo mainrepo_url
     git fetch mainrepo
    
  5. Create a local dev branch
     git checkout -b dev mainrepo/dev
    
  6. Push the local dev to your remote (forked repo):
     git push -u origin dev
    

Create a feature branch

Each task or ticket MUST be developed in a separate branch.
  1. Create a feature branch from the latest stat of the central dev.
     git fetch mainrepo
     git checkout -b featureXXX mainrepo/dev
    
  2. Push the feature branch to your fork and track it
     git push -u origin featureXXX
    

Sync your feature branch

As often as possible sync your feature branch with the central dev.
  1. Sync:
     git fetch mainrepo
     git checkout -f featureXXX
     git rebase mainrepo/dev
    
    • Resolve conflicts if any.
    • Stage each modified file "git add " after conflicts are resolved.
    • You can also use "git checkout --theirs/--ours " to help to resolve conflicts.
    • Use "git rebase --continue" to continue rebasing.
  2. push to you remote clone:
     git push origin
    
    Most probably your local repo and the remote repo will be diverged at this point. Git will warn you that you are about to change the history and will not allow you to push.
    Please, revise the output to make sure that you are actually pushing to the right repo and only after that execute the following to force git to change the history.
     git push -f origin
    
    We recommend to do push in two steps intuitionally to prevent unwanted changes. Even if you are 100% sure, ALWAYS first execute "git push" without "-f". Revise the output. Check that the repo you are pushing is the the one you want and only then force push with "-f".

Request to pull

  1. Always rebase to the main dev before requesting to pull.
     git fetch mainrepo
     git checkout -f featureXXX
     git rebase mainrepo/dev
    
    • Resolve conflicts if any.
    • Stage each modified file "git add " after conflicts are resolved.
    • You can also use "git checkout --theirs/--ours " to help to resolve conflicts.
    • Use "git rebase --continue" to continue rebasing.
  2. Squash all of your commits. Once your code is perfect, clean up its history.
     git rebase -i mainrepo/dev
    
    It is very important for the history of the main repository that all of your commits are squashed. In the future nobody is interested to see your "cosmetic changes" commits or commits related to any other minor changes. The best way to introduce a feature is to introduce it as a patch. This is why, do squash all your commits into one, write a good proper comment before requesting to pull your code.
  3. Push your changes to your remote repo. You may need to use "push -f" since after the rebase your remote repo can be diverged from the local repo.
     git push -f origin
    
  4. Request to pull. Let your release manager know that you want your patch to be merged with the central dev branch. Use a simple email or github to send a pull request.
Stop working on the featureXXX branch, after you sent a request to pull. Create a new branch for any other feature/ticket/bug.

Release manager

We use github as a host for our repository. The write access to the repository is restricted to release managers only. The repository contains the following permanent branches: masterdev. The "RC" and "HotFix" are created (branched) only when we need them and deleted as soon as their commits are merged into the master or when we don't need those branches.
Release managers should merge changes only via a local repo. Don't use github WEBUI or other GUI tools to process pull requests. Why? See below in Process pull requests.

Prepare the environment

  1. setup git configuration. The following you have yo execute on all of your machines, or copy the git config to all of your machines.
     git config --global branch.autosetuprebase always
     git config --global user.name "FirsName LastName"
     git config --global user.email johndoe@example.com
    
  2. Using github fork the main repo.
  3. Create a local copy of the forked repo:
     git clone url_of_the_fork
    
  4. Now when your origin points to your fork. You need to add the main repo to your remotes as well. You should have "origin --> you fork" and "mainrepo --> the main repo".
     git remote add mainrepo mainrepo_url
     git fetch mainrepo
    
  5. create a local dev branch with upstream to the mainrepo/dev.
     git checkout -t -b dev mainrepo/dev
    

Process pull requests

Unfortunately github doesn't support --ff-only merge yet (https://github.com/blog/843-the-merge-button). But it will. The feature is in development. Until the feature is not release you have to process pull requests manually. When a pull request arrives, do the fowling
  1. update
     git fetch origin
     git fetch mainrepo
    
  2. Add the developer's repo to your remotes. You need to do it only once per developer, when you for the first time fetch from this developer
     git remote add dev_name developerrepo_url
     git fetch dev_name
    
  3. Now merge the changes the developer has provided
     git checkout -f dev
     git rebase mainrepo/dev
     git merge --ff-only dev_name/featureXXX
    
    If there are conflicts or git says, that it can't use fast forward, than reject the request and ask developer to rebase from the main dev branchy again, fix conflicts if needed and send a new pull request.
  4. If no conflicts are found, push this commit to the main dev branch:
     git push mainrepo dev:dev
    

Prepare a Release Candidate

Hot Fixes

May 13, 2013

Arrested Development. Season 4.

Anar here...

Great NEWS! Netflix will be streaming the 4th season of the greatest TV show of All Time.


Apr 28, 2013

error MSB4044: The "CppClean" task was not given a value for the required parameter "FoldersToClean"

Anar here...

I know it can't be too late, but just in case you are converting an old VS 2008 project to VS2010 and facing this error when trying to clean your newly converted project:

error MSB4044: The "CppClean" task was not given a value for the required parameter "FoldersToClean"

...than check settings of your project. If the "Output Directory" and/or "Intermediate Directory" contain the "$(ConfigurationName)" macro, than change it to "$(Configuration)".

Simply to say, if one of those those two settings contain a macro, which return an empty or a bad result, the error will pop up.

Hope it helps!

Dec 18, 2012

C++: modules

Anar here...

check it out, LLVM discussing modules in C++: presentation slides