feat(security): add code-scanning with CodeQL#3229
Conversation
24ff606 to
6e16ffe
Compare
Codecov Report
@@ Coverage Diff @@
## main #3229 +/- ##
=======================================
Coverage 46.90% 46.90%
=======================================
Files 23 23
Lines 1196 1196
Branches 237 237
=======================================
Hits 561 561
Misses 451 451
Partials 184 184 Continue to review full report at Codecov.
|
| # make bootstrap | ||
| # make release | ||
|
|
||
| - name: Perform CodeQL Analysis |
There was a problem hiding this comment.
How are we going to handle the existing findings, are we going to fix things first, or merge and then reduce them gradually?
Looks pretty useful though!
Also seems like it might be a good idea to ignore code that we don't control (e.g. lib/vscode) if we can configure things that way
There was a problem hiding this comment.
Probably merge and reduce gradually. Seems like a solid approach.
Good idea on ignoring lib/vscode. I'll see if that's configurable
There was a problem hiding this comment.
I found this in the docs: Specifying Directories to Scan
It's unclear whether this needs to be in a custom codeql config file or if I can put it in the workflow file. I'm going to try adding to the workflow file.
There was a problem hiding this comment.
6e16ffe to
0934acc
Compare
2364f92 to
2bf0907
Compare



This PR adds code-scanning using CodeQL (by GitHub) to help automatically detect common vulnerability and coding errors.
Fixes #3176