-
Notifications
You must be signed in to change notification settings - Fork 7
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
Updating yarn packages #173
Conversation
@whereismyjetpack Note the build failure |
something went to funk-town with this lock file. i modified Dockerfile to run < magic.gif > |
This still isn't quite right. adding app content should always come after a yarn install. this allows the best level of caching for docker builds. I don't think the generated lockfile in this case is quite right. I'm not sure how we got here, but if i clone the repository down and run in a folder with a fresh
the only way to clear it up is to regenerate the lock file and push it up. I'm curious if you could blow away |
f6400ae
to
be6528c
Compare
@whereismyjetpack I don't think we want the integrity check. After rebuilding the lockfile locally with a fresh install of node_modules, the build works. Leaving in the Right now, the build is failing for other reasons: test database isn't available. |
The integrity check is just a safety net. rails runs this anyway, and i'd much rather have the build break, than the deployment. https://yarnpkg.com/lang/en/docs/cli/install/#toc-yarn-install-frozen-lockfile |
Updates everything to the latest, and resolves GHSA-h9rv-jmmf-4pgx related to XSS vulnerabilities with serialize-javascript. Adds --frozen-lockfile option to the Dockerfile so that deploys will fail if yarn updates are needed.
Updates everything to the latest, and resolves GHSA-h9rv-jmmf-4pgx related to XSS vulnerabilities with serialize-javascript.