Skip to content

A few tweaks to be in line with official docs :) - #10

Merged
akanshgulati merged 3 commits into
akanshgulati:masterfrom
patrickhulce:patch-1
Dec 6, 2019
Merged

A few tweaks to be in line with official docs :)#10
akanshgulati merged 3 commits into
akanshgulati:masterfrom
patrickhulce:patch-1

Conversation

@patrickhulce

Copy link
Copy Markdown
Contributor

Feel free to change as needed, explanations inline.

Comment thread src/posts/integrating-lighthouse-ci/integrate-lighthouse-ci.md
Comment thread src/posts/integrating-lighthouse-ci/integrate-lighthouse-ci.md
Comment thread src/posts/integrating-lighthouse-ci/integrate-lighthouse-ci.md
Comment thread src/posts/integrating-lighthouse-ci/integrate-lighthouse-ci.md
Comment thread src/posts/integrating-lighthouse-ci/integrate-lighthouse-ci.md
@akanshgulati

Copy link
Copy Markdown
Owner

@patrickhulce Hi, have you tried to review the comments above ?

@patrickhulce

Copy link
Copy Markdown
Contributor Author

Hi, have you tried to review the comments above ?

I don't see any comments on my view, maybe a pending PR review needs to be submitted on your end?

image

Comment thread src/posts/integrating-lighthouse-ci/integrate-lighthouse-ci.md
Comment thread src/posts/integrating-lighthouse-ci/integrate-lighthouse-ci.md
@akanshgulati

Copy link
Copy Markdown
Owner

Hi, have you tried to review the comments above ?

I don't see any comments on my view, maybe a pending PR review needs to be submitted on your end?

image

@patrickhulce Please check above comment. Sorry, I thought it's submitted.

akanshgulati and others added 2 commits December 7, 2019 03:52
It's mandatory to include 'staticDistDir' field for custom URLs.
@akanshgulati

Copy link
Copy Markdown
Owner

Thanks for reviewing and adding inputs @patrickhulce .

@akanshgulati
akanshgulati merged commit 226a4ab into akanshgulati:master Dec 6, 2019
@patrickhulce
patrickhulce deleted the patch-1 branch December 6, 2019 22:40
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