Hibernating Rhinos | How To Contribute (Git Workflow)
Overview
The general process for working with rhino-esb is:
- Fork on GitHub
- Clone your fork locally
- Create a local branch
git checkout -b myBranch - Work on your feature, spiking/prototyping as required
- Rebase if required (see below)
- Push the branch up to GitHub
git push origin myBranch - Send a Pull Request on GitHub
You should never work on a clone of master, and you should never send a pull request from master - always from a branch. The reasons for this are detailed below.
Please do not include hidden gems (i.e. updated libs, different framework version, etc.) in your pull requests. Our time is just as limited as yours, so we shouldn't have to pick out the relevant pieces.
Handling Updates from Upstream/Master
While you're working away in your branch it's quite possible that your upstream master may be updated. If this happens you should:
- Add upstream to remotes - only needed once
git remote add upstream git://github.com/hibernating-rhinos/rhino-esb.git - Stash any un-committed changes you need to
- Switch to the master branch
git checkout master - Pull latest changes from upstream
git pull upstream master - Switch back to feature branch
git checkout myBranch - Rebase from current master
git rebase master myBranch - Push latest changes to your remote master
git push origin master - (optional) this this makes sure your remote master is up to date This ensures that your history is "clean" i.e. you have one branch off from master followed by your changes in a straight line. Failing to do this ends up with several "messy" merges in your history, which we don't want. This is the reason why you should always work in a branch and you should never be working in, or sending pull requests from, master.
If you're working on a long running feature then you may want to do this quite often, rather than run the risk of potential merge issues further down the line.
Sending a Pull Request
While working on your feature you may well create several branches, which is fine, but before you send a pull request you should ensure that you have rebased back to a single "Feature branch" - we care about your commits, and we care about your feature branch; but we don't care about how many or which branches you created while you were working on it
When you're ready to go you should confirm that you are up to date and rebased with upstream/master (see "Handling Updates from Upstream/Master" above), and then:
- Push your feature branch to origin
git push origin myBranch - Send a descriptive Pull Request on GitHub - making sure you have selected the correct branch in the GitHub UI!
- Wait for Corey or Ayende to merge your changes in.
Original workflow (aka plagiarized from) created for Nancy by TheCodeJunkie and RobertTheGrey. Thanks guys!
For more information on the merits of this workflow please see RobertTheGrey's excellent post on the Fubu MVC Development Group.
Last update: 2/6/2011 7:06:29 PM