kaxil commented on a change in pull request #11211:
URL: https://github.com/apache/airflow/pull/11211#discussion_r498206410
##########
File path: .github/workflows/codeql-analysis.yml
##########
@@ -0,0 +1,66 @@
+name: "CodeQL"
+
+on:
+ push:
+ branches: [master]
+ pull_request:
+ # The branches below must be a subset of the branches above
+ branches: [master]
+ schedule:
+ - cron: '0 17 * * 3'
Review comment:
I mean yeah I don't have a strong opinion on that one tbh. We can keep
it too.
> I think it will just be overwhelming and counter-productive to get those
reports for all commits, but Let's see :)
Wouldn't we get that if we keep "schedule" too, i.e. based on what we have
in PR: we will currently have it when a PR is pushed, a commit is pushed and
"on schedule" too, correct me if I am wrong though
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
