Skip to content

test: faster builds maybe#774

Open
janl wants to merge 7 commits into
masterfrom
test/faster-docker-maybe
Open

test: faster builds maybe#774
janl wants to merge 7 commits into
masterfrom
test/faster-docker-maybe

Conversation

@janl

@janl janl commented May 23, 2018

Copy link
Copy Markdown
Contributor

No description provided.

@janl janl force-pushed the test/faster-docker-maybe branch from 2897ca4 to 2d443c5 Compare June 17, 2018 07:06
@janl

janl commented Jun 17, 2018

Copy link
Copy Markdown
Contributor Author

this re-adds package-lock.json again which reduces nom install time on travis from ~22s to ~10s. Which is a ~5% impact on total runtime.

We should discuss whether we want to add this.

If we do, I’d recommend we all install the npm-merge-driver, that helps with auto-resolving lock file conflicts.

@janl janl requested review from IrinaJay, Realtin and espy July 9, 2018 12:38
@janl

janl commented Jul 9, 2018

Copy link
Copy Markdown
Contributor Author

@espy @hulkoba @Realtin review pls <3

@rmehner

rmehner commented Jul 9, 2018

Copy link
Copy Markdown
Contributor

Random Drive By (I'm working on all my emails right now and this just popped up): If you use a current npm version and you do have a package-lock.json, then you might wanna use npm ci instead of npm install, which will speed up the install even more.

@janl

janl commented Jul 9, 2018

Copy link
Copy Markdown
Contributor Author

@rmehner thanks, yes! :)

This is a one-step-at-a-time thing where I want to bring back package-lock.json :)

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