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.

Sign up for GitHub

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

Jump to bottom

support the /repos/:repo_key/commits/{sha_branch_tag} API #40

Closed
cvrebert opened this issue Nov 13, 2013 · 6 comments
Closed

support the /repos/:repo_key/commits/{sha_branch_tag} API #40

cvrebert opened this issue Nov 13, 2013 · 6 comments
Labels
enhancement

Comments

@cvrebert
Copy link
Contributor

Per #27, to better match GitHub and make our commit-related APIs more featureful.
rajivm added a commit that referenced this issue Dec 10, 2013
@rajivm
first iteration of support for fancier /repos/:repo_key/commits/ API (#…
67538df 
…40)
rajivm added a commit that referenced this issue Dec 10, 2013
@rajivm
add patch text to /repos/:repo_key/commits/ API (#40)
18ee1dc
@rajivm
Copy link
Contributor

rajivm commented Dec 10, 2013

as much of the "single commit" case as makes sense (given our current other features) has been implemented... next up is the commit comparison

@cvrebert
Copy link
Contributor Author

👍

1 similar comment
@dlindquist
Copy link
Member

👍

rajivm added a commit that referenced this issue Dec 13, 2013
@rajivm
first iteration of support for fancier /repos/:repo_key/commits/ API (#…
5c691ad 
…40)
rajivm added a commit that referenced this issue Dec 13, 2013
@rajivm
add patch text to /repos/:repo_key/commits/ API (#40)
869a913
@cvrebert
Copy link
Contributor Author

The final step of this should be to the remove the custom extensions to the /git/commits API (i.e. ?start_sha= and ?ref_name=)

This was referenced Dec 13, 2013
Closed
Closed
@rajivm
Copy link
Contributor

rajivm commented Dec 13, 2013

ah from that perspective we're still missing support for the multiple commits case

@rajivm rajivm mentioned this issue Dec 13, 2013
Open
@cvrebert
Copy link
Contributor Author

Closing as fixed since implemented and unit-tested in the develop branch.

rajivm added a commit that referenced this issue Dec 17, 2013
@rajivm @cvrebert
first iteration of support for fancier /repos/:repo_key/commits/ API (#…
4c67e5f 
…40)
rajivm added a commit that referenced this issue Dec 17, 2013
@rajivm @cvrebert
add patch text to /repos/:repo_key/commits/ API (#40)
4b8f8ae
rajivm added a commit that referenced this issue Dec 17, 2013
@rajivm @cvrebert
first iteration of support for fancier /repos/:repo_key/commits/ API (#…
c11710f 
…40)
rajivm added a commit that referenced this issue Dec 17, 2013
@rajivm @cvrebert
add patch text to /repos/:repo_key/commits/ API (#40)
e15b41b
rajivm added a commit that referenced this issue Dec 17, 2013
@rajivm @cvrebert
add support for fancier /repos/:repo_key/commits/ API (#40)
d271275
rajivm added a commit that referenced this issue Dec 17, 2013
@rajivm @cvrebert
add support for fancier /repos/:repo_key/commits/ API (#40)
764617b
rajivm added a commit that referenced this issue Dec 17, 2013
@rajivm @cvrebert
add support for fancier /repos/:repo_key/commits/ API (#40)
91831df
rajivm added a commit that referenced this issue Dec 20, 2013
@rajivm @cvrebert
add support for fancier /repos/:repo_key/commits/ API (#40)
7a5d593
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@rajivm @dlindquist @cvrebert