-
Notifications
You must be signed in to change notification settings - Fork 3
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 CI config #91
base: lion_484_templates_support_ci_test_pre
Are you sure you want to change the base?
Update CI config #91
Conversation
omrilotan
commented
Aug 30, 2023
•
edited
Loading
edited
- Use a matrix to run the same step with multiple images
- Use the new cimg images instead of the deprecated "circleci" (EOL 2 years ago with node 17)
- Improved use of cache layer and workspace persistence
9006588
to
fab705e
Compare
- persist_to_workspace: | ||
root: . | ||
paths: | ||
- ./node_modules |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If you persist node_modules
to workspace I suppose you don't need to save it to cache anymore?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The cache persists between runs and makes it quicker to run the next commit
Updated the PR's description to reflect the latest changes |