Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

chore: Enable codeql action #638

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

naveensrinivasan
Copy link

This action runs GitHub's industry-leading semantic code analysis engine,
CodeQL, against a repository's source code to find security vulnerabilities.

https://docs.github.com/en/code-security/code-scanning/automatically-scanning-your-code-for-vulnerabilities-and-errors/about-code-scanning-with-codeql

https://github.com/ossf/scorecard/blob/main/docs/checks.md#sast
Signed-off-by: naveensrinivasan <172697+naveensrinivasan@users.noreply.github.com>
@codecov
Copy link

codecov bot commented Aug 24, 2022

Codecov Report

Base: 92.90% // Head: 92.90% // No change to project coverage 👍

Coverage data is based on head (e072cab) compared to base (7594d40).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #638   +/-   ##
=======================================
  Coverage   92.90%   92.90%           
=======================================
  Files         106      106           
  Lines        2438     2438           
  Branches      439      439           
=======================================
  Hits         2265     2265           
  Misses        154      154           
  Partials       19       19           

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

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@shine2lay
Copy link
Member

The code scanning seems to be taking forever, maybe the config is off?

@tenshiAMD
Copy link
Contributor

@shine2lay I think we can merge this one since the scan is completed after that we can fix the issue in this code block

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants