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

refactor buildtest report into class Report #555

Merged

Conversation

shahzebsiddiqui
Copy link
Member

No description provided.

@trafico-bot trafico-bot bot added the 🔍 Ready for Review Pull Request is not reviewed yet label Dec 1, 2020
@codecov
Copy link

codecov bot commented Dec 1, 2020

Codecov Report

Merging #555 (1efbd60) into devel (dbddad0) will increase coverage by 8.77%.
The diff coverage is 62.64%.

Impacted file tree graph

@@            Coverage Diff             @@
##            devel     #555      +/-   ##
==========================================
+ Coverage   57.69%   66.46%   +8.77%     
==========================================
  Files          21       26       +5     
  Lines        1553     2454     +901     
  Branches      226      454     +228     
==========================================
+ Hits          896     1631     +735     
- Misses        606      731     +125     
- Partials       51       92      +41     
Impacted Files Coverage Δ
buildtest/log.py 63.16% <ø> (ø)
buildtest/menu/inspect.py 11.11% <11.11%> (ø)
buildtest/menu/compilers.py 12.62% <12.62%> (ø)
buildtest/executors/slurm.py 13.18% <13.18%> (ø)
buildtest/executors/cobalt.py 15.45% <15.45%> (ø)
buildtest/executors/lsf.py 15.60% <15.60%> (ø)
buildtest/executors/setup.py 42.53% <42.53%> (ø)
buildtest/docs.py 60.00% <60.00%> (ø)
buildtest/buildsystem/batch.py 63.79% <63.79%> (ø)
buildtest/menu/schema.py 58.00% <67.86%> (-30.89%) ⬇️
... and 32 more

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 4ead9cc...6219f74. Read the comment docs.

@shahzebsiddiqui shahzebsiddiqui merged commit c54a32a into buildtesters:devel Dec 1, 2020
@trafico-bot trafico-bot bot added ✨ Merged Pull Request has been merged successfully and removed 🔍 Ready for Review Pull Request is not reviewed yet labels Dec 1, 2020
@shahzebsiddiqui shahzebsiddiqui deleted the report_refactor_class branch December 8, 2020 18:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ Merged Pull Request has been merged successfully size/L
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant