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 #14

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

Conversation

WindzCUHK
Copy link
Contributor

@WindzCUHK WindzCUHK commented Jan 10, 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]>
Signed-off-by: wfan <[email protected]>
@codecov-commenter
Copy link

codecov-commenter commented Jan 10, 2023

Codecov Report

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

@@            Coverage Diff            @@
##             master      #14   +/-   ##
=========================================
  Coverage          ?   88.50%           
=========================================
  Files             ?       17           
  Lines             ?     1209           
  Branches          ?        0           
=========================================
  Hits              ?     1070           
  Misses            ?      123           
  Partials          ?       16           
Flag Coverage Δ
unittests 88.50% <0.00%> (?)

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

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

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