Skip to content

Add lighthouse ci#2

Open
jcs98 wants to merge 1 commit intomasterfrom
jcs98/lighthouse-stats
Open

Add lighthouse ci#2
jcs98 wants to merge 1 commit intomasterfrom
jcs98/lighthouse-stats

Conversation

@jcs98
Copy link
Owner

@jcs98 jcs98 commented Nov 11, 2020

Motivation

(Write your motivation here.)

Have you read the Contributing Guidelines on pull requests?

(Write your answer here.)

Test Plan

(Write your test plan here. If you changed any code, please provide us with clear instructions on how you verified your changes work. Bonus points for screenshots and videos!)

Related PRs

(If this PR adds or changes functionality, please take some time to update the docs at https://github.com/facebook/docusaurus, and link to your PR here.)

@jcs98 jcs98 force-pushed the jcs98/lighthouse-stats branch 3 times, most recently from 70d604a to 669cc68 Compare November 12, 2020 10:23
@jcs98 jcs98 force-pushed the jcs98/lighthouse-stats branch from 669cc68 to de3ad04 Compare November 12, 2020 10:47
{
"ci": {
"collect": {
"staticDistDir": "./website/build"

Choose a reason for hiding this comment

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

Instead of adding a staticDistDir. I believe this should be done

"startServerCommmand" : "yarn start",
"url" : [The URL being output by Webpack]

Copy link
Owner Author

Choose a reason for hiding this comment

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

Do you know what is the "The URL being output by Webpack" ? I'm not sure about that.
Also yarn start will start the site on the dev server which I don't believe will give the same lighthouse scores as the actual build.

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