-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Introduce GitHub Code Scanning #1117
Conversation
Signed-off-by: Tom Kerkhove <kerkhove.tom@gmail.com>
7241d96
to
2eef55f
Compare
Have you been thinking in incorporating this under https://github.com/kedacore/keda/blob/v2/.github/workflows/pr-validation.yml ? |
Honestly, no. Just wanted to keep it separate but happy to merge if you prefer! |
I'll move it! |
YOu should have a look at how many I have here 😂 tomkerkhove/promitor#1265 |
No way man 😄 I got your point, sometimes the break down is good. But for the stuff you have added, the |
Signed-off-by: Tom Kerkhove <kerkhove.tom@gmail.com>
Signed-off-by: Tom Kerkhove <kerkhove.tom@gmail.com>
Signed-off-by: Tom Kerkhove <kerkhove.tom@gmail.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.
LGTM
Introduce GitHub Code Scanning.
I'm not using Snyk here since this is built-in but if we want to use that instead (given it will do all our container scanning) I'm happy to change it.