-
Notifications
You must be signed in to change notification settings - Fork 2
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
E: update .github pipelines dependencies #47
Conversation
3e45c23
to
d7ca3d3
Compare
bfdfbf9
to
4ce8c68
Compare
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.
Node 20 is the current LTS baseline. Otherwise good. Thanks!
uses: actions/cache@v3 | ||
env: | ||
cache-name: cache-node-modules | ||
node-version: '18' |
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.
Why not 20?
uses: actions/cache@v3 | ||
env: | ||
cache-name: cache-node-modules | ||
node-version: '18' |
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.
Why not 20?
- name: Set up Node | ||
uses: actions/setup-node@v4 | ||
with: | ||
node-version: '18' |
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.
Same
No description provided.