I wrote them as class methods because to me they didn't seem to need to need any context, so I didn't see why instantiating an object to do that stuff.

I tried to google the matter, [this](http://nicksda.apotomo.de/2011/07/are-class-methods-evil/), and [this](http://mlomnicki.com/programming/ruby/2011/07/20/class-vs-instance-methods.html) are interesting, but I'm still not 100% sold...

Can you guys suggest other reading?

Still it's true that `GitHelper` has all instance methods, so for the sake of consistency I'll fix this.