should we keep codeql in every push to pull requests & main? #822
Closed
jeromedockes
started this conversation in
General
Replies: 2 comments
-
I'm +1 to remove it for now and think about scheduling later. |
Beta Was this translation helpful? Give feedback.
0 replies
-
Close by removing CodeQL |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
We have a workflow running github's codeQL which apparently does:
(I haven't looked at it in depth)
scikit-learn, scipy, polars, joblib, pytest, arrow, cloudpickle, cryptography, flask don't seem to use it but apparently numpy, pandas, and matplotib do
it is one of the checks that take a long time in the CI; maybe we can remove it or at least run it only with a scheduled job and not every push to a PR branch or main?
Beta Was this translation helpful? Give feedback.
All reactions