Skip to content
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

Added apis to list commits by author and list the repos of an organization (including private ones) #77

Closed
wants to merge 4 commits into from

Conversation

vr100
Copy link
Contributor

@vr100 vr100 commented Mar 19, 2014

Hi,

I have added apis for the following

  • List the repos of organization (including the private ones). The existing implementation does not list the private repos of an organization even with the appropriate scope permissions.
  • List the commits by author and branch. [Note that both branch and sha are given as sha parameters to the url. I verified this manually. Got this solution from stackoverflow : http://stackoverflow.com/questions/9179828/github-api-retrieve-all-commits-for-all-branches-for-a-repo

Thank you,
vr100

@buildhive
Copy link

Kohsuke Kawaguchi » github-api #140 SUCCESS
This pull request looks good
(what's this?)

@buildhive
Copy link

Kohsuke Kawaguchi » github-api #141 SUCCESS
This pull request looks good
(what's this?)

@buildhive
Copy link

Kohsuke Kawaguchi » github-api #142 SUCCESS
This pull request looks good
(what's this?)

@buildhive
Copy link

Kohsuke Kawaguchi » github-api #143 SUCCESS
This pull request looks good
(what's this?)

@kohsuke
Copy link
Collaborator

kohsuke commented Mar 28, 2014

Thanks for the pull request!

Given that the commit search API has multiple independent criteria, we should define a builder pattern to build up the search criteria. Would you be interested in reworking it a little bit toward that direction?

@kohsuke
Copy link
Collaborator

kohsuke commented Apr 19, 2014

No reaction from the original author. Closing it for my book-keeping simplicity. Feel free to reopen.

@kohsuke kohsuke closed this Apr 19, 2014
@vr100
Copy link
Contributor Author

vr100 commented Apr 22, 2014

Sorry. I missed the message. I would be interested in reworking on this. This weekend I will work on this and send in a pull request.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants