Skip to content

Upgrade dependencies, add debug logs #46

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

Merged
merged 5 commits into from
Nov 26, 2022

Conversation

roryabraham
Copy link
Contributor

@roryabraham roryabraham commented Nov 2, 2022

Hello 👋🏼

We love this action, but we saw some errors with it today. I forked the action and was able to fix it. I'm not sure exactly which of these changes resolved the error(s), but they all seem reasonably beneficial.

  • Switch from Node 12 to Node 16
  • Upgrade all dependencies to latest
  • Fix some resultant TypeScript errors
  • Add debug logs

@fbuinosquy1985
Copy link

Please merge and release it.

@roryabraham
Copy link
Contributor Author

If you need this fix, you can use my fork in the meantime: https://github.com/roryabraham/turnstyle

@michaelstepner
Copy link

FYI @fbuinosquy1985 and @roryabraham:

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.

@fbuinosquy1985
Copy link

FYI @fbuinosquy1985 and @roryabraham:

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.

yes i know that, but i want to use this anyway

@michaelstepner
Copy link

yes i know that, but i want to use this anyway

👍, i expect some other people looking at this PR might not know this package duplicates now-native functionality, so hopefully my comment will be useful to others.

@dudicoco
Copy link

dudicoco commented Nov 25, 2022

FYI @fbuinosquy1985 and @roryabraham:

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 that's incorrect, the native functionality only waits for one job to complete while this action acts as a queue waiting for all previous jobs to complete.

Per the docs:

When a concurrent job or workflow is queued, if another job or workflow using the same concurrency group in the repository is in progress, the queued job or workflow will be pending. Any previously pending job or workflow in the concurrency group will be canceled.

@michaelstepner
Copy link

the native functionality only waits for one job to complete while this action acts as a queue waiting for all previous jobs to complete.

Good point @dudicoco! It would be more accurate to say that concurrency is not a complete replacement for turnstyle:

  • There are features in turnstyle not present in concurrency
  • There are features in concurrency not present in turnstyle

@softprops softprops merged commit ca99add into softprops:master Nov 26, 2022
@varunaroratest
Copy link

Hi @softprops
Please create a release with the latest changes.

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.

6 participants