Goify consts #39

Merged
carlosmn merged 2 commits from goify-consts into master 2013-09-17 10:03:07 -05:00
carlosmn commented 2013-09-12 03:52:46 -05:00 (Migrated from github.com)

This allows for better typing and nicer-looking code.


I'm not sure what to do with the error consts though; maybe we shouldn't be exposing them at all and work more like other libraries and return the same error for a particular type (not found, etc.). I'm leaving them capitalized here.

This allows for better typing and nicer-looking code. --- I'm not sure what to do with the error consts though; maybe we shouldn't be exposing them at all and work more like other libraries and return the same error for a particular type (not found, etc.). I'm leaving them capitalized here.
vmg commented 2013-09-17 10:03:03 -05:00 (Migrated from github.com)

Yeah, this one looks very good. :)

Yeah, this one looks very good. :)
Sign in to join this conversation.
No reviewers
No Milestone
No project
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: jcarr/git2go#39
No description provided.