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

fix: usage of node12 which is deprecated #521

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

hamirmahal
Copy link
Contributor

fixes #520.

Problem

tests.yml uses node12 which is deprecated.

How this change fixes it

This updates a dependency 2 layers deep that uses node12 which is deprecated.

runs:
using: composite
steps:
- uses: actions/setup-node@v2
- uses: actions/setup-node@v4
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

v4 is the latest major version of actions/setup-node at the time of this writing.

@vitonsky
Copy link
Collaborator

@hamirmahal why actions did not start in this PR?

image

@hamirmahal
Copy link
Contributor Author

I'm not sure, but all checks passed on my fork.

image

@vitonsky vitonsky enabled auto-merge November 21, 2024 17:01
@vitonsky vitonsky disabled auto-merge November 22, 2024 00:37
@vitonsky
Copy link
Collaborator

Could you update branch and push again. Maybe this problem will be fixed then

@hamirmahal hamirmahal force-pushed the fix/usage-of-node12-which-is-deprecated branch from 7559214 to 048cea9 Compare December 13, 2024 18:58
@hamirmahal
Copy link
Contributor Author

Okay, I just did.

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.

tests.yml uses node12 which is deprecated
2 participants