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

remove developer preview warning from .combined_status() #505

Conversation

theckman
Copy link
Contributor

@theckman theckman commented Sep 1, 2014

The combined_status API endpoint is no longer in development preview. This patch makes the following changes:

  • removes the HTTP Accept header that was required for this call while it was in its preview period
  • remove the warning printed about the method being in the development preview

All tests passed locally when following the steps in the contribution information. Please let me know if there are any changes that are needed to be made.

The combined_status API endpoint is no longer in development preview. This patch makes the following changes:

* removes the HTTP `Accept` header that was required for this call while it was in its preview period
* remove the warning printed about the method being in the development preview

All tests passed locally when following the steps in the contribution information. Please let me know if there are any changes that are needed to be made.
pengwynn added a commit that referenced this pull request Sep 5, 2014
…bined_status

remove developer preview warning from .combined_status()
@pengwynn pengwynn merged commit 506ade9 into octokit:master Sep 5, 2014
@pengwynn
Copy link
Collaborator

pengwynn commented Sep 5, 2014

🙇

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