17 Jul 2025 |
Mic92 | Probably gonna split this into smaller prs | 12:13:32 |
Sergei Zimmerman (xokdvium) | Do we want the checks in CI to land after the fixes? Did we actually agree on an appropriate set of checks? | 12:29:31 |
Mic92 | I think we want them in CI. Takes a while to scan. | 13:21:38 |
Sergei Zimmerman (xokdvium) | I've tried cooking up yesterday. Nixpkgs also has clang-tidy-sarif to output sarif traces, that can be published to the PR as annotations via the codeql-action/upload-sarif@v1 action. Maybe that would work well with combination with git-clang-tidy? | 13:22:58 |
Mic92 | So you mean you only want to scan affected files? Good idea | 13:29:35 |
Mic92 | * So you mean you only want to scan changed files? Good idea | 13:29:42 |
Mic92 | Than we can enable even more checks. | 13:30:28 |