Skip to content

Conversation

@dplewis
Copy link
Member

@dplewis dplewis commented May 10, 2019

prepare causes npm ci to be used. .npm is the recommended cache directory for npm ci. Should speed up build times.

https://docs.npmjs.com/cli/ci

`prepare` causes `npm ci` to be used. `.npm` is the recommended cache directory. Should speed up build times.

https://docs.npmjs.com/cli/ci
@codecov
Copy link

codecov bot commented May 10, 2019

Codecov Report

Merging #5575 into master will decrease coverage by 0.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #5575      +/-   ##
==========================================
- Coverage   94.03%   94.02%   -0.02%     
==========================================
  Files         129      129              
  Lines        9194     9194              
==========================================
- Hits         8646     8645       -1     
- Misses        548      549       +1
Impacted Files Coverage Δ
src/RestWrite.js 93.36% <0%> (-0.35%) ⬇️
...dapters/Storage/Postgres/PostgresStorageAdapter.js 96.9% <0%> (+0.08%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8a543bf...1f931c6. Read the comment docs.

@dplewis dplewis requested a review from acinader May 10, 2019 01:43
@dplewis dplewis merged commit 4c996ba into master May 10, 2019
@dplewis dplewis deleted the travis-cache branch May 10, 2019 03:04
@dplewis
Copy link
Member Author

dplewis commented May 10, 2019

@acinader You can probably speed up test by a minute or two if you remove the jasmine spec reporter. Just FYI

UnderratedDev pushed a commit to UnderratedDev/parse-server that referenced this pull request Mar 21, 2020
`prepare` causes `npm ci` to be used. `.npm` is the recommended cache directory. Should speed up build times.

https://docs.npmjs.com/cli/ci
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.

3 participants