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.

Sign up for GitHub

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

Jump to bottom

coverage tool support #107

Merged
merged 10 commits into from
Sep 30, 2022
Merged

coverage tool support #107

merged 10 commits into from
Sep 30, 2022

Conversation

wbpcode
Copy link
Contributor

@wbpcode wbpcode commented Sep 7, 2022

Signed-off-by: wangbaiping wbphub@live.com

@wbpcode
coverage tool support
cf0c72d 
Signed-off-by: wangbaiping <wbphub@live.com>
@wbpcode
Copy link
Contributor Author

wbpcode commented Sep 7, 2022

Run the coverage.sh script in the workspace dir.

@wbpcode
Copy link
Contributor Author

wbpcode commented Sep 7, 2022 

Overall coverage rate:
  lines......: 68.2% (696 of 1020 lines)
  functions..: 58.8% (107 of 182 functions)
HTML coverage report is in ./coverage_report//index.html
All coverage report files are in ./coverage_report//cpp2sky_coverage.tar.gz

@wu-sheng wu-sheng added this to the 0.5.1 milestone Sep 7, 2022
wu-sheng
wu-sheng previously approved these changes Sep 7, 2022
View reviewed changes
@wu-sheng wu-sheng removed this from the 0.5.1 milestone Sep 7, 2022
@wbpcode
reset env
910189b 
Signed-off-by: wangbaiping <wbphub@live.com>
@wbpcode
clang is necessary for coverage report
2f722da 
Signed-off-by: wangbaiping <wbphub@live.com>
@wbpcode
comment all jobs except coverage for test
7f0269e 
Signed-off-by: wangbaiping <wbphub@live.com>
@codecov
Copy link

codecov bot commented Sep 29, 2022 •
edited
Loading

Codecov Report

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

@@           Coverage Diff           @@
##             main     #107   +/-   ##
=======================================
  Coverage        ?   67.67%           
=======================================
  Files           ?       26           
  Lines           ?     1021           
  Branches        ?        0           
=======================================
  Hits            ?      691           
  Misses          ?      330           
  Partials        ?        0

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@wbpcode
recover all ci because coverage is done :)
040447f 
Signed-off-by: wangbaiping <wbphub@live.com>
@wbpcode
fix job's name
3b791fe 
Signed-off-by: wangbaiping <wbphub@live.com>
@wbpcode
Copy link
Contributor Author

wbpcode commented Sep 30, 2022 •
edited
Loading

cc @wu-sheng This is done.

@wu-sheng wu-sheng added this to the 0.6.0 milestone Sep 30, 2022
@wu-sheng wu-sheng merged commit bd7a7e5 into SkyAPM:main Sep 30, 2022
@wbpcode wbpcode mentioned this pull request Oct 7, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@wu-sheng wu-sheng wu-sheng approved these changes

Assignees
No one assigned
Labels
testing
Projects
None yet
Milestone
0.6.0
Development

Successfully merging this pull request may close these issues.
2 participants
@wbpcode @wu-sheng