-
Notifications
You must be signed in to change notification settings - Fork 137
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
ci: exclude codeql on yaml #1008
Conversation
Signed-off-by: Asra Ali <asraa@google.com>
74c7e00
to
1e1e740
Compare
Signed-off-by: Asra Ali <asraa@google.com>
Signed-off-by: Asra Ali <asraa@google.com>
TBD I'll make a skipping job or figure out something else. |
Signed-off-by: Asra Ali <asraa@google.com>
Signed-off-by: Asra Ali <asraa@google.com>
Updated because GitHub recommends creating a duplicate workflow with the same name and job that runs on the ignored paths and performs a successful noop |
@asraa I removed CodeQL as a required check. After this is merged we can hopefully add "CodeQL / Analyze" to the required checks on the protected branch rules. |
Signed-off-by: Asra Ali <asraa@google.com>
Fixes #1047
I used the guidance on https://docs.github.com/en/repositories/configuring-branches-and-merges-in-your-repository/defining-the-mergeability-of-pull-requests/troubleshooting-required-status-checks#handling-skipped-but-required-checks to create a no-op job with the same name.
Signed-off-by: Asra Ali asraa@google.com