Ensure workflows only run for wolfssl repository_owne#811
Ensure workflows only run for wolfssl repository_owne#811gojimmypi wants to merge 1 commit intowolfSSL:masterfrom
Conversation
|
What if we added limits to which branches the test would be run on to 'master', 'main', or 'release/*'? |
|
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? |
|
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. |
|
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. :) |
|
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. |
|
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