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.

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

@vitonsky
Copy link
Member

@hamirmahal we should run CI before merge, but CI does not run

image

I'm not sure in reason, but we can try to fix it. Could you rebase branch to latest master.

In commit 75f3f35 i've updated a CI hooks, so CI must run fine now.

@hamirmahal hamirmahal force-pushed the fix/usage-of-node12-which-is-deprecated branch from 048cea9 to b842564 Compare January 26, 2026 18:04
@vitonsky vitonsky changed the title fix: usage of node12 which is deprecated ci: use node v20 Jan 26, 2026
@vitonsky vitonsky merged commit d496d94 into translate-tools:master Jan 26, 2026
4 checks passed
@vitonsky
Copy link
Member

@hamirmahal CI fixed, thank you for contribution

@hamirmahal
Copy link
Contributor Author

You're welcome!

@hamirmahal hamirmahal deleted the fix/usage-of-node12-which-is-deprecated branch January 27, 2026 02:13
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