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 scoverage aggregator and fail on low coverage. #1181

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
3 participants
@fnothaft
Member

fnothaft commented Sep 21, 2016

Small Scoverage update --> sets aggregate to true, which produces a report under ${ADAM_HOME}/target at the end of the build containing the aggregated coverage across all submodules.

@AmplabJenkins

This comment has been minimized.

Show comment
Hide comment
@AmplabJenkins

AmplabJenkins Sep 21, 2016

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/ADAM-prb/1499/
Test PASSed.

AmplabJenkins commented Sep 21, 2016

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/ADAM-prb/1499/
Test PASSed.

@heuermh

LGTM

@heuermh

This comment has been minimized.

Show comment
Hide comment
@heuermh

heuermh Sep 21, 2016

Member

Merged as commit d914ee4.

Thank you, @fnothaft!

Member

heuermh commented Sep 21, 2016

Merged as commit d914ee4.

Thank you, @fnothaft!

@heuermh heuermh closed this Sep 21, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment