Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@yorkie
Copy link
Contributor

@yorkie yorkie commented Sep 24, 2015

Hooks API has direct full_name to be respond, so adding a way to
initistant the repo object with full_name should be more convinent.

Now we are able to get a repo:

githubapi.getRepo('nodejs/node')

Thanks

Hooks API has direct full_name to be respond, so adding a way to
initistant the repo object with full_name should be more convinent.
@EliCDavis
Copy link

👍 , would have helped a lot on my project earlier 😅

@yorkie
Copy link
Contributor Author

yorkie commented Sep 24, 2015

Hey @EliCDavis if you hope directly apply my changes into your project, you are able to use https://github.com/weflex/octop for now, because I found the repo under DevelopmentSeed still not active and with so many PR and issues without any feedback. Because I will be working on Github API, so I think Octop might be more active during this time, BTW, of course hope this would be merged as well :-)

@aendra-rininsland
Copy link
Member

This looks like a great commit. Thanks for adding unit tests! Merging for 0.10.7!

@aendra-rininsland aendra-rininsland added this to the v0.10.7 milestone Oct 17, 2015
aendra-rininsland pushed a commit that referenced this pull request Oct 17, 2015
@aendra-rininsland aendra-rininsland merged commit e195ae5 into github-tools:master Oct 17, 2015
@yorkie yorkie deleted the add/new-repo-from-fullname branch October 19, 2015 02:42
@yorkie
Copy link
Contributor Author

yorkie commented Oct 19, 2015

:-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants