-
Notifications
You must be signed in to change notification settings - Fork 2.1k
Update GitHub Pages API to add new response parameters #2109
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
Update GitHub Pages API to add new response parameters #2109
Conversation
Codecov Report
@@ Coverage Diff @@
## master #2109 +/- ##
=======================================
Coverage 97.77% 97.78%
=======================================
Files 111 112 +1
Lines 9934 9956 +22
=======================================
+ Hits 9713 9735 +22
Misses 154 154
Partials 67 67
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you, @rreichel3!
Just a few changes, please, and then we should be ready for a second LGTM before merging.
Co-authored-by: Glenn Lewis <6598971+gmlewis@users.noreply.github.com>
…/rreichel3/go-github into users/rreichel3/update-pages-api
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you, @rreichel3 !
LGTM.
Awaiting second LGTM before merging.
👋 @wesleimp Just wanted to bump this! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
thanks for this addition
Thank you, @cpanato ! |
This PR updates the GitHub Pages API structs to support the latest documented response from GitHub.
This fixes #2107
Its based upon the documentation on GitHub's public docs here and enables support for viewing a GitHub Pages' visibility and HTTPS configuration.