Uh oh!
There was an error while loading. Please reload this page.
Ensure workflows only run for wolfssl repository_owne - #811
Conversation
ejohnstown
commented
Jun 30, 2025
What if we added limits to which branches the test would be run on to 'master', 'main', or 'release/*'? |
gojimmypi
commented
Jul 1, 2025
Instead of the In addition? That might be a good idea if for some reason the repository itself has branches where we don't want the job to run. Does that occur? Generally a PR comes from a user fork, not a local branch, eh? |
ejohnstown
commented
Jul 8, 2025
I think the better solution is to disable running the actions in your own fork. The option is in your repo's Settings:Actions:General. I want the actions to run automatically whenever I push a commit to a branch. |
gojimmypi
commented
Jul 8, 2025
Well, the best solution is to have all the actions disabled by default. Bit that's not what GitHub is doing today. If you, as a maintainer want the actions to always run for you, I could see that being part of the condition. I do not believe that all the workflows should run for all the users in all the forks and all the branches for every change. Just my suggestion - if you prefer to close without merging. :) |
ejohnstown
commented
Jul 8, 2025
And I think it should be left to the repo owner to disable actions if they want to. The only way to enable them with this PR is to edit the actions as part of the branch so they run, then take the changes out after. With the repo setting, it's just a click. |
ejohnstown
commented
Jul 8, 2025
I believe this should be solved by turning off the actions in your own repo's setting rather than affecting everyone. I don't think adding an My two-bits. |
Description
Same as wolfSSL/wolfssl#8936, but for wolfSSH:
Continues the pattern of running wolfssl GitHub workflows only on wolfssl repository owner:
At one point, it was only annoying when some of the job would fail in my fork.
While viewing my GitHub Billing, I noticed a recent increase in "charges" related to workflows that run on my fork of wolfssl when I add commits.
I'm not sure if this is a new feature, perhaps something coming soon, that might have an adverse affect on forks that do not have free tiers. In any case, it is a waste of computing resources to blindly run all of the workflows in a fork.
If desired, they can be manually enabled.
See also: wolfSSL/wolfssl#7871
Fixes zd# n/a
Testing
How did you test?
"tested" on my branch only (to confirm the workflows didn't run)
Checklist