-
-
Notifications
You must be signed in to change notification settings - Fork 156
ci: add CodeQL security scanning #844
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
base: main
Are you sure you want to change the base?
Conversation
Codecov Report
@@ Coverage Diff @@
## main #844 +/- ##
=======================================
Coverage 96.12% 96.12%
=======================================
Files 31 31
Lines 929 930 +1
=======================================
+ Hits 893 894 +1
Misses 36 36
Continue to review full report at Codecov.
|
|
It is not clear to me what CodeQL scanning can do for CITGM. |
|
@targos Well, it used to be known as LGTM, but now it's part of GitHub and continues to improve in open place. |
Signed-off-by: Sora Morimoto <[email protected]>
|
Hey all, I made a boo boo on main and had to force push. I've rebased this branch and force pushed to make sure that you don't have to do extra work because of my mistake |
|
There is no problem at all. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #844 +/- ##
=======================================
Coverage 95.63% 95.63%
=======================================
Files 31 31
Lines 940 940
=======================================
Hits 899 899
Misses 41 41 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
As the title says.
Checklist
npm testpasseshere