citgm icon indicating copy to clipboard operation
citgm copied to clipboard

ci: add CodeQL security scanning

Open smorimoto opened this issue 3 years ago • 5 comments

As the title says.

Checklist
  • [ ] npm test passes
  • [ ] tests are included
  • [ ] documentation is changed or added
  • [ ] contribution guidelines followed here

smorimoto avatar Jan 16 '21 04:01 smorimoto

Codecov Report

Merging #844 (222bcdb) into main (e1716d6) will increase coverage by 0.00%. The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #844   +/-   ##
=======================================
  Coverage   96.12%   96.12%           
=======================================
  Files          31       31           
  Lines         929      930    +1     
=======================================
+ Hits          893      894    +1     
  Misses         36       36           
Impacted Files Coverage Δ
lib/grab-project.js 92.00% <0.00%> (+0.16%) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update e1716d6...222bcdb. Read the comment docs.

codecov-io avatar Jan 16 '21 04:01 codecov-io

It is not clear to me what CodeQL scanning can do for CITGM.

targos avatar Mar 05 '21 10:03 targos

@targos Well, it used to be known as LGTM, but now it's part of GitHub and continues to improve in open place.

smorimoto avatar Mar 10 '21 18:03 smorimoto

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

MylesBorins avatar Mar 18 '21 21:03 MylesBorins

There is no problem at all.

smorimoto avatar Mar 18 '21 23:03 smorimoto