-
-
Notifications
You must be signed in to change notification settings - Fork 61
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
Conversation
Please merge and release it. |
If you need this fix, you can use my fork in the meantime: https://github.com/roryabraham/turnstyle |
FYI @fbuinosquy1985 and @roryabraham: Github has incorporated equivalent functionality to 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 |
👍, 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. |
@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:
|
Good point @dudicoco! It would be more accurate to say that
|
Hi @softprops |
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.