Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Usually, my Cocoa-fu says that convenience initializers get declared as class methods, designated initializers go under -init.... Arguably, this one could be kept as a designated initializer (since it ends up calling one of the handful libgit2 functions that creates objects), but our designated initializer is actually initWithGitRepository:.

I'm not sure how that would like after being Swift-preprocessed though.