-
Notifications
You must be signed in to change notification settings - Fork 107
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
Address CodeQL warnings #565
Comments
Thanks, how is the CodeQL report generated? Since CodeQL is enabled, I was expecting the results to be part of the current checks but, for example at: https://github.com/linux-system-roles/network/actions/runs/4138105148 |
We ported the command lines that appeared in the codeql action to https://github.com/linux-system-roles/tox-lsr/blob/main/src/tox_lsr/test_scripts/runcodeql.sh. Actually, the result contains warnings in addition to the failures. And the network role is free from "failures". |
Ah, thanks for the clarification. Now I found the warnings at https://github.com/linux-system-roles/network/security/code-scanning |
@tyll, I've opened a pr for the tox command It clearly states as follows.
|
CodeQL reports following warnings. It'd be preferable to clean them up.
codeql.network.txt
This is an incomplete pull request containing useful discussions.
#558
The text was updated successfully, but these errors were encountered: