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
disable codeql on terraform-only changes #751
Conversation
Signed-off-by: Bob Callaway <bcallaway@google.com>
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.
sorry double check that
@@ -7,6 +7,8 @@ on: | |||
pull_request: | |||
branches: | |||
- main | |||
paths-ignore: | |||
- 'terraform/**' |
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.
humm that will not run and the check that is required will not be valid and will block
we should use the dorny/paths-filter@v2
to filter then that will properly skip the job
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.
hmm, its worked across other workflows in this repo....... I guess I don't understand the difference?
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.
Oh really? That is good news then GitHub changed that behavior nice
This reverts commit 4d6984b. Branch protection rules require the CodeQL check. Signed-off-by: Cody Soyland <codysoyland@github.com>
* Revert "disable codeql on terraform-only changes (#751)" This reverts commit 4d6984b. Branch protection rules require the CodeQL check. Signed-off-by: Cody Soyland <codysoyland@github.com> * Filter paths for CodeQL Signed-off-by: Cody Soyland <codysoyland@github.com> --------- Signed-off-by: Cody Soyland <codysoyland@github.com>
Summary
Release Note
Documentation