Skip to content

Node 12 deprecated #43

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

Closed
hkroger opened this issue Oct 26, 2022 · 6 comments
Closed

Node 12 deprecated #43

hkroger opened this issue Oct 26, 2022 · 6 comments

Comments

@hkroger
Copy link

hkroger commented Oct 26, 2022

Should this be updated to use node 16 as github is suggesting?

https://github.blog/changelog/2022-09-22-github-actions-all-actions-will-begin-running-on-node16-instead-of-node12/

@fbuinosquy1985
Copy link

Please merge it.

@michaelstepner
Copy link

FYI, Github has incorporated equivalent functionality to turnstyle into the Github Actions spec using the 'concurrency' parameter. softprops/turnstyle was developed in 2020 and Github added the concurrency parameter in 2021.

The native 'concurrency' implementation has additional features beyond what's possible in turnstyle, and it is lower cost: it leaves jobs pending rather than running while they wait for a previous job to end.

@jeherve
Copy link

jeherve commented Feb 1, 2023

I commented on the atttached PR but wanted to post this here as well for extra visibility.

While the concurrency parameter is useful, I'm afraid there are still situations1 where it isn't enough to catch potential duplicated runs. In those situations, softprops/turnstyle can still be useful.

Thanks!

Footnotes

  1. For reference, here is one such case.

@anomiex
Copy link
Contributor

anomiex commented Feb 1, 2023

More generally, GitHub's concurrency parameter handles the cases where you want existing runs (queued and optional the in-progress run too) to be cancelled when a new run comes in, but does not (yet) handle cases where you want runs to wait for in-progress runs without any being cancelled.

See https://github.com/orgs/community/discussions/5435 for one discussion asking for the feature (with GH replying "we hope to get to this someday").

@daimenworrall-cinch
Copy link

Hello! Are there any updates on this issue? Thanks!

@chenrui333
Copy link
Collaborator

Close as fixed. Thanks!

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 a pull request may close this issue.

7 participants