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

Trigger creation of a code coverage report via GitHub Actions #371

Closed
ximon18 opened this issue Jan 12, 2021 · 2 comments · Fixed by #372
Closed

Trigger creation of a code coverage report via GitHub Actions #371

ximon18 opened this issue Jan 12, 2021 · 2 comments · Fixed by #372

Comments

@ximon18
Copy link
Member

ximon18 commented Jan 12, 2021

E.g. use cargo tarpaulin triggered after a successful CI build.
ximon18 added a commit that referenced this issue Jan 12, 2021
@ximon18
Initial attempt at code coverage using tarpaulin (#371)
56c44c4
ximon18 added a commit that referenced this issue Jan 12, 2021
@ximon18
Extend the CI job with code coverage reports by cargo tarpaulin. (#371)
990b5c2
ximon18 added a commit that referenced this issue Jan 12, 2021
@ximon18
Extend the CI job with code coverage reports by cargo tarpaulin. (#371)
74da0c2
@ximon18 ximon18 linked a pull request Jan 12, 2021 that will close this issue
Extend the CI job with code coverage reports by cargo tarpaulin. (#371) #372
Merged
@ximon18
Copy link
Member Author

ximon18 commented Jan 12, 2021

Linked pull request #372 creates a cargo tarpaulin code coverage HTML report and uploads it to GitHub as a workflow artifact. Code coverage is reported both for default features and for all features. In the image below you can see hte difference in the report for default features vs all features at the top level of the source tree:

image

These reports can be drilled into, e.g.:

image

@ximon18 ximon18 mentioned this issue Jan 12, 2021
Merged
ximon18 added a commit that referenced this issue Jan 14, 2021
@ximon18
Print out the GH Actions event and env details to find out how to tri…
29490ea 
…gger it on the same branch and commit as the workflow that we triggered on. (#371)
ximon18 added a commit that referenced this issue Jan 14, 2021
@ximon18
Add missing @ref part of GH Actions action specifier. (#371)
d5e7813
ximon18 added a commit that referenced this issue Jan 14, 2021
@ximon18
Add _correct_ missing @ref part of GH Actions action specifier. (#371)
9bd7d23
ximon18 added a commit that referenced this issue Jan 14, 2021
@ximon18
Attempt to checkout the correct code to cover. (#371)
eff17f7
ximon18 added a commit that referenced this issue Jan 14, 2021
@ximon18
YML syntax correction. (#371)
90013b8
ximon18 added a commit that referenced this issue Jan 14, 2021
@ximon18
YML syntax correction. (#371)
780a3d3
ximon18 added a commit that referenced this issue Jan 14, 2021
@ximon18
Remove coverage.yml from this branch as the master branch copy is the…
7d3cb81 
… one GH is using. (#371)
ximon18 added a commit that referenced this issue Jan 14, 2021
@ximon18
Update coverage.yml to hopefully checkout the right sources. (#371)
3685445
ximon18 added a commit that referenced this issue Jan 14, 2021
@ximon18
Put the coverage workflow file back in the branch so that it is part …
26dd1a7 
…of the PR review. (#371)
ximon18 added a commit that referenced this issue Jan 15, 2021
@ximon18
Extend the CI job with code coverage reports by cargo tarpaulin. (#371)…
8a7a7bc 
… (#372)

Trigger a new code coverage report GitHub Actions CI workflow after successful completion of the CI workflow (#371).
Also attempts to stablize the flakey add ROAs UI test (#377).
@ximon18
Copy link
Member Author

ximon18 commented Jan 29, 2021

Closing as all new successful (and test) runs on the dev branch now cause a coverage report job to run.

@ximon18 ximon18 closed this as completed Jan 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Extend the CI job with code coverage reports by cargo tarpaulin. (#371) NLnetLabs/krill
1 participant
@ximon18