Skip to content

Add sha parameter to Pull Request merge #381

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
merged 1 commit into from
Jul 9, 2015

Conversation

drobertduke
Copy link

The Github v3 API supports an optional 'sha' parameter to explicitly specify the commit that should be at the head of a Pull Request when merging it.
https://developer.github.com/v3/pulls/#merge-a-pull-request-merge-button

@sigmavirus24
Copy link
Owner

Thanks @drobertduke! Are you working with the 0.9.x or 1.0.0 alpha version of github3.py? This belongs in both branches, but I wanted to know if you need a 0.9.x release really soon or not.

The Github v3 API supports an optional 'sha' parameter to explicitly
specify the commit that should be at the head of a Pull Request
when merging it.
https://developer.github.com/v3/pulls/#merge-a-pull-request-merge-button
@drobertduke
Copy link
Author

@sigmavirus24 I'm working with the alpha version, I don't think I'll need a 0.9.x release soon. Thanks! I can submit this PR to the stable branch also.

@sigmavirus24
Copy link
Owner

I thought I had merged this already!

sigmavirus24 added a commit that referenced this pull request Jul 9, 2015
Add sha parameter to Pull Request merge
@sigmavirus24 sigmavirus24 merged commit 52e1bae into sigmavirus24:develop Jul 9, 2015
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.

2 participants