Skip to content

[ci][python] Add coverage check in CI#6861

Merged
CalvinKirs merged 6 commits intoapache:devfrom
zhongjiajie:ci-add-coverage
Nov 17, 2021
Merged

[ci][python] Add coverage check in CI#6861
CalvinKirs merged 6 commits intoapache:devfrom
zhongjiajie:ci-add-coverage

Conversation

@zhongjiajie
Copy link
Copy Markdown
Member

@zhongjiajie zhongjiajie commented Nov 16, 2021

Add test coverage for our CI, closes: #6870

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Nov 16, 2021

Codecov Report

Merging #6861 (3944a4e) into dev (94352a4) will increase coverage by 0.04%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##                dev    #6861      +/-   ##
============================================
+ Coverage     33.07%   33.12%   +0.04%     
  Complexity     1619     1619              
============================================
  Files           433      433              
  Lines         14292    14292              
  Branches       1427     1427              
============================================
+ Hits           4727     4734       +7     
+ Misses         9116     9107       -9     
- Partials        449      451       +2     
Impacted Files Coverage Δ
...lphinscheduler/rpc/common/AbstractRpcCallBack.java 0.00% <0.00%> (-100.00%) ⬇️
...r/plugin/registry/zookeeper/ZookeeperRegistry.java 54.54% <0.00%> (+7.27%) ⬆️

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 94352a4...3944a4e. Read the comment docs.

@zhongjiajie zhongjiajie changed the title [ci] Add coverage check in CI [WIP][ci][python] Add coverage check in CI Nov 16, 2021
@zhongjiajie zhongjiajie changed the title [WIP][ci][python] Add coverage check in CI [ci][python] Add coverage check in CI Nov 16, 2021
@sonarqubecloud
Copy link
Copy Markdown

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

Copy link
Copy Markdown
Member

@CalvinKirs CalvinKirs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@CalvinKirs CalvinKirs merged commit 54933b3 into apache:dev Nov 17, 2021
@zhongjiajie zhongjiajie deleted the ci-add-coverage branch November 17, 2021 02:05
@zhongjiajie
Copy link
Copy Markdown
Member Author

Thanks~

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.

[Feature][python] Add unit test coverage minimum threshold to python API

3 participants