turnstyle icon indicating copy to clipboard operation
turnstyle copied to clipboard

Node 12 deprecated

Open hkroger opened this issue 3 years ago • 5 comments

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/

hkroger avatar Oct 26 '22 06:10 hkroger

Please merge it.

fbuinosquy1985 avatar Nov 03 '22 12:11 fbuinosquy1985

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.

michaelstepner avatar Nov 25 '22 14:11 michaelstepner

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 situations[^1] where it isn't enough to catch potential duplicated runs. In those situations, softprops/turnstyle can still be useful.

Thanks!

[^1]: For reference, here is one such case.

jeherve avatar Feb 01 '23 18:02 jeherve

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").

anomiex avatar Feb 01 '23 19:02 anomiex

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

daimenworrall-cinch avatar Feb 08 '23 13:02 daimenworrall-cinch

Close as fixed. Thanks!

chenrui333 avatar Jun 19 '24 23:06 chenrui333