Skip to content

Add DNS health checks support for GitHub Pages - #2746

Merged
gmlewis merged 3 commits into
google:masterfrom
denist11:add_pages_health_check_support
Apr 12, 2023
Merged

Add DNS health checks support for GitHub Pages#2746
gmlewis merged 3 commits into
google:masterfrom
denist11:add_pages_health_check_support

Conversation

@denist11

Copy link
Copy Markdown
Contributor

Fixes#2745

Allows requesting a DNS health check for the CNAME configured for a repository's GitHub Pages.

@gmlewisgmlewis left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thank you, @oslowalk !
Just a few minor tweaks, please.

Comment threadgithub/repos_pages.go Outdated
Comment threadgithub/repos_pages.go Outdated
Comment threadgithub/repos_pages.go Outdated
@gmlewisgmlewis added the NeedsReview PR is awaiting a review before merging. label Apr 9, 2023
@codecov

codecovBot commented Apr 9, 2023

Copy link
Copy Markdown

Codecov Report

Merging #2746 (bc30782) into master (4fec23d) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@ Coverage Diff @@## master #2746 +/- ##
=======================================
Coverage 98.05% 98.05% =======================================
Files 132 132 Lines 11611 11623 +12 =======================================
+ Hits 11385 11397 +12 
Misses 154 154 Partials 72 72 
Impacted FilesCoverage Δ
github/repos_pages.go97.43% <100.00%> (+0.29%)⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@gmlewisgmlewis left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thank you, @oslowalk !
LGTM.

Awaiting second LGTM+Approval from any other contributor to this repo before merging.

@valbeatvalbeat left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM!

@gmlewisgmlewis changed the title Add DNS health checks support for Github PagesAdd DNS health checks support for GitHub PagesApr 12, 2023
@gmlewisgmlewis removed the NeedsReview PR is awaiting a review before merging. label Apr 12, 2023
@gmlewis

Copy link
Copy Markdown
Collaborator

Thank you, @valbeat !
Merging.

@gmlewis
gmlewis merged commit 683c2d7 into google:masterApr 12, 2023
jlaportebot added a commit to jlaportebot/go-github that referenced this pull request Jun 28, 2026
Sign up for freeto 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.

Add support for DNS health checks on Pages.

3 participants

@denist11@gmlewis@valbeat