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

Update npm, test Node.js 9, detect package-lock churn in CI #1601

Merged
merged 4 commits into from Nov 28, 2017

Conversation

Projects
None yet
2 participants
@novemberborn
Member

novemberborn commented Nov 28, 2017

npm 5.6.0 means we can now safely test with Node.js 9, so I've added that to the test matrix.

I've also added a way of verifying the package-lock.json file is up to date. This should help detect unexpected or missing changes in future pull requests and reduce any churn to just when the npm version itself is updated. See https://travis-ci.org/avajs/ava/jobs/308555431 for an example of when package-lock.json is outdated.

novemberborn added some commits Nov 28, 2017

@sindresorhus sindresorhus merged commit 4124d77 into master Nov 28, 2017

6 of 7 checks passed

dependency-ci Checking Dependencies
Details
codecov/patch Coverage not affected when comparing e401bd1...06827b3
Details
codecov/project 97.55% remains the same compared to e401bd1
Details
continuous-integration/appveyor/branch AppVeyor build succeeded
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details

@sindresorhus sindresorhus deleted the node-npm-ci-bump branch Nov 28, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment