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

implement commit by sha endpoint for gogs #7

Closed
bradrydzewski opened this issue Dec 17, 2018 · 5 comments · Fixed by #20
Closed

implement commit by sha endpoint for gogs #7

bradrydzewski opened this issue Dec 17, 2018 · 5 comments · Fixed by #20

Comments

@bradrydzewski
Copy link
Member

bradrydzewski commented Dec 17, 2018

It looks like gogs added the commit endpoint in the latest version:
https://github.com/gogs/docs-api/blob/master/Repositories/Commits.md

This needs to be integrated here:
https://github.com/drone/go-scm/blob/master/scm/driver/gogs/git.go#L26

@ogarcia
Copy link
Contributor

ogarcia commented Jan 18, 2019

@bradrydzewski Any update about this?

@bradrydzewski
Copy link
Member Author

bradrydzewski commented Jan 18, 2019

No updates from me. The fastest way to get this issue resolved would be to create (and test) a patch and then submit a pull request. I have a pretty big backlog of items I'm working through right now, and this is not currently a top priority for me.

@lflare
Copy link

lflare commented Feb 19, 2019

@bradrydzewski I'm guessing this is still at the bottom of your backlog?

@tboerger
Copy link
Contributor

For gitea there had been a pr merged, but for Gogs nobody provided one.

@bradrydzewski
Copy link
Member Author

I am going to mark this as help wanted, and will accept a pull request for this item.

@drone drone locked and limited conversation to collaborators Feb 19, 2019
bradrydzewski added a commit that referenced this issue Apr 10, 2019
Fetch specific commit by sha for gogs, closes #7
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants