Skip to content

Conversation

patrickmarabeas
Copy link
Contributor

@patrickmarabeas patrickmarabeas commented Mar 5, 2020

Implements support for updating Github Pages sites - allowing CName and
Source to be customized.

Fixes: #1361

Implements support for updating Github Pages sites - allowing CName and
Source to be customized.
@googlebot googlebot added the cla: yes Indication that the PR author has signed a Google Contributor License Agreement. label Mar 5, 2020
Copy link
Collaborator

@gmlewis gmlewis left a comment

Choose a reason for hiding this comment

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

Thank you, @patrickmarabeas!
Just a few changes, please.

Please make sure to run:

go generate ./...
go test ./...
go vet ./...

and commit and push the results (no force push, please... we squash the commits anyway).

Copy link
Collaborator

@gmlewis gmlewis left a comment

Choose a reason for hiding this comment

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

Thank you, @patrickmarabeas!
This looks great.

Awaiting second LGTM before merging.

@gmlewis gmlewis requested a review from gauntface March 5, 2020 04:50
Copy link
Collaborator

@wesleimp wesleimp left a comment

Choose a reason for hiding this comment

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

LGTM 👌

@gmlewis
Copy link
Collaborator

gmlewis commented Mar 9, 2020

Thank you, @wesleimp! Merging.

@gmlewis gmlewis merged commit 89b491a into google:master Mar 9, 2020
sprsld pushed a commit to sprsld/go-github that referenced this pull request Mar 24, 2020
n1lesh pushed a commit to n1lesh/go-github that referenced this pull request Oct 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes Indication that the PR author has signed a Google Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

How to add custom domain to Github Page ?

4 participants