Skip to content

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.

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
Member

@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
Member

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