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
Update yarn to 1.16.0 #29503
Update yarn to 1.16.0 #29503
Conversation
Sorry I missed this PR earlier, Josh. I would suggest updating the docs here to use Lines 13 to 20 in e654149
or alternately pointing people to SETUP.md if you don't want duplication. |
Also removes the explict `npm rebuild node-sass` because it's now part of post-install.
The parts I commented on look good! |
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.
Thanks for doing all the docker image updates as well!
&& echo "ALTER USER 'root'@'localhost' IDENTIFIED WITH mysql_native_password BY '';" | mysql \ | ||
&& chown -R mysql:mysql /var/lib/mysql /var/run/mysqld \ |
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.
Nitpick: I don't think you need to run the same chown
command twice since the changes to the directories will persist from the first time.
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.
I tried without first, but I was getting a hang on shutdown. Any idea why that would be?
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.
🤷♀yea I don't know then.
Patterned after PR #22093. Replaces PR #29495.
I will send mail separately asking folks to upgrade their local install of yarn.