Skip to content

use npm ci for CA#2730

Merged
emmachughes merged 1 commit intomasterfrom
ca-use-npm-ci
May 30, 2024
Merged

use npm ci for CA#2730
emmachughes merged 1 commit intomasterfrom
ca-use-npm-ci

Conversation

@emmachughes
Copy link
Contributor

@emmachughes emmachughes commented May 30, 2024

npm install alters package.json, which is undesirable.

https://docs.npmjs.com/cli/v10/commands/npm-ci

--legacy-peer-deps isn't necessary since 1a8424e, but that commit did not remove it from docker-compose.yml.

@emmachughes emmachughes requested a review from chrieinv May 30, 2024 09:46
@emmachughes emmachughes merged commit 2755c45 into master May 30, 2024
@emmachughes emmachughes deleted the ca-use-npm-ci branch May 30, 2024 09:53
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