How do you feel about the "Error " prefix being redundant?
This is a style thing but I prefer to leave it out, so for example
this would say

```go
log.Fatalf("cloning repo to %s on branch %s: %s\n", dir, buildbranch, err)
```