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 xml report for codecov #1966

Merged
merged 2 commits into from
Mar 18, 2021
Merged

add xml report for codecov #1966

merged 2 commits into from
Mar 18, 2021

Conversation

pxLi
Copy link
Collaborator

@pxLi pxLi commented Mar 18, 2021

Signed-off-by: Peixin Li peixinl@nvidia.com

requested by the team, better to have a fancy codecov report, e.g.
https://app.codecov.io/gh/pxLi/spark-rapids/branch/add-codecov-support
coverage report for the HEAD branch (login w/ github account to see details on each files)

If this one looks OK. I will setup this into our nightly pipeline to report daily for our dev branch,
NOTE: we will not integrate this with our pre-merge for now

Signed-off-by: Peixin Li <peixinl@nvidia.com>
@pxLi pxLi added the build Related to CI / CD or cleanly building label Mar 18, 2021
@pxLi
Copy link
Collaborator Author

pxLi commented Mar 18, 2021

build

@pxLi
Copy link
Collaborator Author

pxLi commented Mar 18, 2021

build

@pxLi pxLi merged commit aacd517 into NVIDIA:branch-0.5 Mar 18, 2021
@sameerz sameerz added this to the Mar 15 - March 26 milestone Mar 22, 2021
nartal1 pushed a commit to nartal1/spark-rapids that referenced this pull request Jun 9, 2021
* add xml report for codecov

Signed-off-by: Peixin Li <peixinl@nvidia.com>

* update copyright

Co-authored-by: Peixin Li <peixinl@nvidia.com>
nartal1 pushed a commit to nartal1/spark-rapids that referenced this pull request Jun 9, 2021
* add xml report for codecov

Signed-off-by: Peixin Li <peixinl@nvidia.com>

* update copyright

Co-authored-by: Peixin Li <peixinl@nvidia.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build Related to CI / CD or cleanly building
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants