We don't seem to be handling failure here.  We could use something more robust like https://github.com/mortardata/mortar/blob/master/lib/mortar/git.rb#L52 and check for failure.