Skip to content
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

Cache node_modules instead of the NPM cache #755

Merged
merged 1 commit into from
Nov 16, 2017
Merged

Conversation

malept
Copy link
Member

@malept malept commented Nov 16, 2017

  • I have read the contribution documentation for this project.
  • I agree to follow the code of conduct that this project follows, as appropriate.
  • The changes are appropriately documented (not applicable).
  • The changes have sufficient test coverage (not applicable).
  • The testsuite passes successfully on my local machine (not applicable).

Summarize your changes:

Hopefully this drops the CI cache size (and build time) while also making sure that it gets tested with the latest dependencies.

@malept malept merged commit 1975ee8 into master Nov 16, 2017
@malept malept deleted the better-ci-caching branch November 16, 2017 06:50
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.

None yet

1 participant