Skip to content
This repository was archived by the owner on Aug 26, 2021. It is now read-only.

Conversation

azerella
Copy link
Contributor

@azerella azerella commented Sep 4, 2018

The CI/CD pipeline from feature/zdt-test brach doesn't have caching functionality, despite it being supported. The caching support provided by CircleCI is limited as of 09/18:

  • No support for save_cache key path globbing.
  • No support for multiple checksums as key names
  • No support for checksum on directories
  • Cannot export {{ .ENVIRONMENT.variableName }} using an eval expression such as $(ls)

This PR is a temporary workaround to enable caching. It will require the user to run the md5sum command listed in CONTRIBUTING.md anytime a new module is installed or modified assuming checksums of the package-lock.json

Quick caching tests show:

  • With cache: ~7 mins
  • Without cache: ~12 mins

@azerella azerella changed the base branch from master to develop September 4, 2018 01:05
@alex-page alex-page added the parked Won’t work on anytime soon as low priority. label Sep 4, 2018
@alex-page
Copy link
Contributor

alex-page commented Sep 4, 2018

I am going to add this to the icebox until circleci updates how this works. Currently we have a lot of processes to follow when deploying and I don't want to add another just yet.

Another option is moving to yarn with something like bolt. That would make this process a lot easier.

@azerella
Copy link
Contributor Author

azerella commented Feb 8, 2019

Closing this in favour of #603.

Using yarn with a single hoisted node_modules folder makes circleci caching way easier 💯

@azerella azerella closed this Feb 8, 2019
@azerella azerella deleted the feature/circleci-cache-node-modules branch February 8, 2019 00:35
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
parked Won’t work on anytime soon as low priority.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants