wolfssl
wolfssl copied to clipboard
Add conditional repository_owner to workflow
Description
This PR adds a workflow condition to only run the workflow on wolfssl-owned repositories (in this case, async.yml). I propose doing this on all workflows, but this PR is only for the first one to open a discussion and code review. I'm happy to make the changes for others.
Why?
I've recently begun implementing automated Espressif CI testing. One of the things that has been annoying and even problematic has been all the upstream workflows running in my fork, such as this sync from upstream:
Worse, some of them appear to never stop:
Although I think that's actually a GitHub bug, as the ones from last month are not actually running:
Fixes zd# n/a
Testing
How did you test?
not tested.
Checklist
- [ ] added tests
- [ ] updated/added doxygen
- [ ] updated appropriate READMEs
- [ ] Updated manual and documentation
Update: thanks @julek-wolfssl for the tip on manually disabling workflows in a fork. That works, but I found it rather unintuitive.
I still think it's a good idea to not run all the upstream workflows by default. We could have contributors that may be time-limited on computing power or something. It just seems to be a waste. Thoughts?
Jenkins retest this please
Conditional Updates
After a discussion with @bandi13 I've updated all the wolfSSL workflows to conditionally run on:
if: github.repository_owner == 'wolfssl'
This condition was generally placed between the name: and runs-on: keywords.
socat_check update
I noticed a syntax highlighting error for the socat_check strategy (none was defined), so I removed it.
@bandi13 please review these changes to github action tests