Skip to content

Create codeql-analysis.yml#7057

Closed
chrisbra wants to merge 3 commits intomasterfrom
chrisbra-patch-1
Closed

Create codeql-analysis.yml#7057
chrisbra wants to merge 3 commits intomasterfrom
chrisbra-patch-1

Conversation

@chrisbra
Copy link
Member

@chrisbra chrisbra commented Oct 1, 2020

Enable Code scanning for security vulnerabilities.

Github just announced to allow automatic code scanning using Code-QL here: https://github.blog/2020-09-30-code-scanning-is-now-available/

So let's just set this up and see if this works (or if it causes too many problems and false positives we can disable it again).

Enable Code scanning for security vulnerabilities.

Github just announced to allow automatic code scanning using Code-QL here: https://github.blog/2020-09-30-code-scanning-is-now-available/

So let's just set this up and see if this works (or if it causes too many problems and false positives we can disable it again).
@codecov
Copy link

codecov bot commented Oct 1, 2020

Codecov Report

Merging #7057 into master will increase coverage by 0.18%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #7057      +/-   ##
==========================================
+ Coverage   88.46%   88.64%   +0.18%     
==========================================
  Files         148      148              
  Lines      158938   161760    +2822     
==========================================
+ Hits       140605   143395    +2790     
- Misses      18333    18365      +32     
Impacted Files Coverage Δ
src/session.c 82.00% <0.00%> (-1.54%) ⬇️
src/xdiff/xemit.c 67.10% <0.00%> (-1.39%) ⬇️
src/libvterm/src/vterm.c 66.66% <0.00%> (-0.79%) ⬇️
src/xdiff/xpatience.c 89.22% <0.00%> (-0.59%) ⬇️
src/libvterm/src/screen.c 59.35% <0.00%> (-0.46%) ⬇️
src/hardcopy.c 83.95% <0.00%> (-0.36%) ⬇️
src/ex_cmds.c 91.42% <0.00%> (-0.33%) ⬇️
src/version.c 92.13% <0.00%> (-0.31%) ⬇️
src/if_perl.xs 86.08% <0.00%> (-0.30%) ⬇️
src/gui_xim.c 23.75% <0.00%> (-0.24%) ⬇️
... and 121 more

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 5366e1a...20903b0. Read the comment docs.

@brammool
Copy link
Contributor

brammool commented Oct 1, 2020

Looks like you need to remove "javascript".

remove javascript
@brammool
Copy link
Contributor

brammool commented Oct 1, 2020

Can you re-trigger CI? Patch 8.2.1783 should fix the failures. I wonder if I could trigger this somehow...

@brammool brammool closed this in fa79be6 Oct 2, 2020
@k-takata k-takata deleted the chrisbra-patch-1 branch October 2, 2020 08:32
janlazo added a commit to janlazo/neovim that referenced this pull request Oct 3, 2020
Problem:    Crash with 'incsearch' and very long line.
Solution:   Check whether regprog becomes NULL. (closes vim/vim#7063)
vim/vim@795aaa1

N/A patches for version.c:

vim-patch:8.2.1784: commits are not scanned for security problems

Problem:    commits are not scanned for security problems
Solution:   Enable Github code scanning. (Christian Brabandt, closes vim/vim#7057)
vim/vim@fa79be6
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants