-
Notifications
You must be signed in to change notification settings - Fork 7
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
Updated workflow #11
Comments
What about committing to a specific That aside, ideally this workflow would update the existing "new release" issue if one is already open, rather than creating another. Might be an edge case, but YOURLS wouldn't be the first project to publish multiple versions in quick succession after a packaging mishap or other oversight makes a version unusable for some reason. No idea if that's actually happened in recent YOURLS history (don't think so), but planning ahead is 👌. |
Yes that would be a very valid option, yet a bit out of the scope of my documentation for which I use your plugin as a test case My goal is to provide a simple workflow example, and indeed my previous assumptions were that simply committing would trigger tests, which turned out false, just like filing a PR as you suggest (by design, workflows can't trigger workflow, you have to work around with apps or personal tokens) So, to sum up, just pointing out that the workflow as I initially wrote it won't do much :) |
Sigh. I understand why GitHub did things this way, but it's still frustrating. Feel free to make a PR with whatever solution you think is best. If you can't do the "update existing issue" part, don't worry about it. |
In my opinion given the pace of development of YOURLS and most plugins, the best is the simple workflow I described above. (Can't PR, I deleted my fork) |
(Fork again?) |
I came up with a better workflow for
check-yourls-release.yml
:It :
Updated workflow :
Cheers !
The text was updated successfully, but these errors were encountered: