-
Notifications
You must be signed in to change notification settings - Fork 120
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
[develop] Update GitHub Actions workflows to run on specific branches #412
[develop] Update GitHub Actions workflows to run on specific branches #412
Conversation
branches: | ||
- develop | ||
- 'release/*' | ||
workflow_dispatch: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Might as well add a manual trigger to this one, too.
@christinaholtNOAA Can we remove |
@danielabdi-noaa I get a bunch of those, too. I'm curious what others think. It looks like @mark-a-potts and @willmayfield were involved in introducing that file, so I'd like to get their input on its removal. I'm happy to make the change, if we decide on it. |
It is fine with me to remove it. |
DESCRIPTION OF CHANGES:
GitHub Actions workflows keep trying to run on my forks when I push new branches. Many times I'm not ready to run the tests when I'm backing up my branches.
I think it's a better use of resources to run those tests when a PR is open, and when that PR is merged, or a push directly to a set of specific branches occurs. The goal of this code change the behavior of the GHA workflows to do just that.
Type of change
TESTS CONDUCTED:
These changes aren't relevant to anything other than the GHA test workflows, and can only be tested via this PR. Fingers crossed!
DEPENDENCIES:
DOCUMENTATION:
ISSUE:
CHECKLIST
LABELS (optional):
A Code Manager needs to add the following labels to this PR: