Skip to content

Use npm ci for fast(er) installs on CI#6292

Merged
kumavis merged 1 commit intoMetaMask:developfrom
whymarrh:npm-ci
Mar 12, 2019
Merged

Use npm ci for fast(er) installs on CI#6292
kumavis merged 1 commit intoMetaMask:developfrom
whymarrh:npm-ci

Conversation

@whymarrh
Copy link
Copy Markdown
Contributor

This PR updates our CI config to use the npm ci command to speed up our CI installs step.

This command is similar to npm-install, except it’s meant to be used in automated environments such as test platforms, continuous integration, and deployment – or any situation where you want to make sure you’re doing a clean install of your dependencies. It can be significantly faster than a regular npm install by skipping certain user-oriented features. It is also more strict than a regular install, which can help catch errors or inconsistencies caused by the incrementally-installed local environments of most npm users.

  • If a node_modules is already present, it will be automatically removed before npm ci begins its install.

Which is fine now that we're not caching it anymore.

@whymarrh whymarrh requested a review from kumavis March 12, 2019 15:43
@whymarrh
Copy link
Copy Markdown
Contributor Author

Before:[1]

added 4269 packages from 2533 contributors in 131.205s

After:[2]

added 4274 packages in 70.026s

@kumavis kumavis merged commit 9d130a1 into MetaMask:develop Mar 12, 2019
@whymarrh whymarrh deleted the npm-ci branch March 12, 2019 16:08
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