Skip to content

ci: add codeql analysis #597

Merged
JoanFM merged 1 commit intomainfrom
feat-ci-add-securitiy-scan
Oct 7, 2022
Merged

ci: add codeql analysis #597
JoanFM merged 1 commit intomainfrom
feat-ci-add-securitiy-scan

Conversation

@samsja
Copy link
Copy Markdown
Member

@samsja samsja commented Oct 7, 2022

Signed-off-by: samsja [email protected]

Context

Adding a security scan on our codebase.

Note this is the default one propose by github

@samsja samsja force-pushed the feat-ci-add-securitiy-scan branch from 134dc06 to 53c27c2 Compare October 7, 2022 10:18
@samsja samsja changed the title Create codeql-analysis.yml feat(ci): add codeql analysis Oct 7, 2022
@samsja samsja requested review from JoanFM and alaeddine-13 October 7, 2022 10:28
@codecov
Copy link
Copy Markdown

codecov bot commented Oct 7, 2022

Codecov Report

Merging #597 (53c27c2) into main (376e9fa) will decrease coverage by 1.77%.
The diff coverage is 42.10%.

@@            Coverage Diff             @@
##             main     #597      +/-   ##
==========================================
- Coverage   84.92%   83.15%   -1.78%     
==========================================
  Files         133      133              
  Lines        6708     6714       +6     
==========================================
- Hits         5697     5583     -114     
- Misses       1011     1131     +120     
Flag Coverage Δ
docarray 83.15% <42.10%> (-1.78%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
docarray/array/mixins/find.py 88.65% <0.00%> (ø)
docarray/array/queryset/lookup.py 78.26% <0.00%> (ø)
docarray/array/queryset/parser.py 87.09% <0.00%> (ø)
docarray/array/storage/annlite/helper.py 50.00% <0.00%> (ø)
docarray/document/generators.py 78.63% <0.00%> (-7.70%) ⬇️
docarray/document/mixins/plot.py 78.91% <0.00%> (ø)
docarray/document/mixins/porting.py 93.15% <0.00%> (ø)
docarray/dataclasses/setter.py 92.15% <33.33%> (-3.68%) ⬇️
docarray/dataclasses/types.py 91.37% <50.00%> (ø)
docarray/document/mixins/multimodal.py 93.33% <50.00%> (ø)
... and 22 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@samsja samsja mentioned this pull request Oct 7, 2022
4 tasks
@JoanFM JoanFM changed the title feat(ci): add codeql analysis ci: add codeql analysis Oct 7, 2022
@JoanFM JoanFM merged commit a3d5de1 into main Oct 7, 2022
@JoanFM JoanFM deleted the feat-ci-add-securitiy-scan branch October 7, 2022 14:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants