-
Notifications
You must be signed in to change notification settings - Fork 9.3k
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
Migrate all dev scripts + docs from npm to yarn. #2072
Conversation
.travis.yml
Outdated
before_script: | ||
- gem install travis-artifacts | ||
- export DISPLAY=:99.0 | ||
- export LIGHTHOUSE_CHROMIUM_PATH="$(pwd)/chrome-linux/chrome" | ||
- sh -e /etc/init.d/xvfb start | ||
- ./lighthouse-core/scripts/download-chrome.sh | ||
- npm run build-all | ||
- yarn run build-all |
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.
Pretty sure the "run" is optional.
yarn blah
would be great for all the things.
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.
ya it is. i was keeping it in some places to be explicit, but really i'd prefer to go without "run", too. :)
i'll nuke them.
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.
done
Users can install install with npm, though
@@ -238,13 +238,12 @@ right corner and signing in to GitHub. | |||
git clone https://github.com/GoogleChrome/lighthouse | |||
|
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.
we should add a note about needing yarn
for development here
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.
sg. done 2bb9c5b
Users can install install with npm, though
Users are very welcome to install lighthouse with npm, but if they're doing a clone and any local development, all scripts + docs use yarn.