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

Allow more deployment statuses per Github docs #1074

Merged

Conversation

greggilbert
Copy link
Contributor

The Github Docs list more acceptable deployment statuses, so this updates the create_status function to allow those.

Copy link
Collaborator

@staticdev staticdev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@greggilbert thanks for this PR. Would you please add your name to contributor's list and also the changes to changelog according to the instructions on https://github.com/sigmavirus24/github3.py/blob/main/CONTRIBUTING.rst

@greggilbert
Copy link
Contributor Author

@staticdev Thanks for the note - I took a swing at it. Let me know!

@staticdev
Copy link
Collaborator

Thanks @greggilbert, now you need to run the test suite again since the formatting of the files you changed are now broken. Sorry for the inconvenience.

@staticdev staticdev added the enhancement New feature or request label Feb 15, 2022
@greggilbert greggilbert force-pushed the feature/add-deployment-statuses branch from 9cd6440 to ec906d0 Compare February 15, 2022 19:42
@greggilbert greggilbert force-pushed the feature/add-deployment-statuses branch from 37fb37c to f4314e8 Compare February 15, 2022 19:46
@greggilbert
Copy link
Contributor Author

@staticdev All good, hopefully I got it right this time.

@sigmavirus24
Copy link
Owner

@staticdev pre-commit CI should auto-fix things it can for us

@sigmavirus24 sigmavirus24 merged commit 6db9296 into sigmavirus24:main Feb 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants