Skip to content
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

add github action for coverage, lint and vulnerability #7

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

Conversation

WindzCUHK
Copy link
Contributor

@WindzCUHK WindzCUHK commented Jan 22, 2023

Signed-off-by: wfan [email protected]

Description

  • add github action for coverage, lint and vulnerability
  • fix lint problems

Type of change

  • Bug fix
  • New feature
  • Refactoring (no functional changes, no api changes)
  • Non-code changes (update documentation, pipeline, etc.)

Flags

  • Breaks backward compatibility
  • Requires a documentation update
  • Has untestable code

Checklist

  • Followed the guidelines in the CONTRIBUTING document
  • Added prefix [skip ci]/[ci skip]/[no ci]/[skip actions]/[actions skip] in the PR title if necessary
  • Tested and linted the code
  • Commented the code
  • Made corresponding changes to the documentation
  • Passed all pipeline checking

Checklist for maintainer

  • Use Squash and merge
  • Double-confirm the merge message has prefix [skip ci]/[ci skip]/[no ci]/[skip actions]/[actions skip]
  • Delete the branch after merge

Signed-off-by: wfan <[email protected]>
@codecov-commenter
Copy link

codecov-commenter commented Jan 22, 2023

Codecov Report

❗ No coverage uploaded for pull request base (master@3b00963). Click here to learn what that means.
The diff coverage is n/a.

❗ Current head fc486e8 differs from pull request most recent head d929119. Consider uploading reports for the commit d929119 to get more accurate results

@@            Coverage Diff            @@
##             master       #7   +/-   ##
=========================================
  Coverage          ?   92.73%           
=========================================
  Files             ?       15           
  Lines             ?     1556           
  Branches          ?        0           
=========================================
  Hits              ?     1443           
  Misses            ?       89           
  Partials          ?       24           
Flag Coverage Δ
unittests 92.73% <0.00%> (?)

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

Signed-off-by: wfan <[email protected]>
Signed-off-by: wfan <[email protected]>
@WindzCUHK WindzCUHK changed the title add lint add github action for coverage, lint and vulnerability Jan 22, 2023
@github-advanced-security
Copy link

This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation.

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