+
Skip to content

repo: add getRepo with fullname #223

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

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
点击 这是indexloc提供的php浏览器服务,不要输入任何密码和下载