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

Travis: config.mk user config; enable coverage #986

Merged
merged 3 commits into from
Aug 30, 2019

Conversation

marshallward
Copy link
Collaborator

This PR enables coverage cleanup and CodeCov upload for test runs.

It also introduces an optional configuration file, config.mk which can
be used for user-configured settings, such as templates or mpirun
commands.

This PR enables coverage cleanup and CodeCov upload for test runs.

It also introduces an optional configuration file, config.mk which can
be used for user-configured settings, such as templates or mpirun
commands.
Path rules for cleaning up old code coverage files (*.gcda) and for
running the CodeCov.io upload script have been fixed.
@codecov-io
Copy link

codecov-io commented Aug 29, 2019

Codecov Report

❗ No coverage uploaded for pull request base (dev/gfdl@9547fce). Click here to learn what that means.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##             dev/gfdl     #986   +/-   ##
===========================================
  Coverage            ?   43.17%           
===========================================
  Files               ?      213           
  Lines               ?    62219           
  Branches            ?        0           
===========================================
  Hits                ?    26866           
  Misses              ?    35353           
  Partials            ?        0

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 9547fce...d496ee7. Read the comment docs.

@marshallward
Copy link
Collaborator Author

This looks like it's working now.

Copy link
Collaborator

@adcroft adcroft left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Since this only affects Travis-CI I'll not submit this to the gaea pipeline.

@adcroft adcroft merged commit b1d59c6 into mom-ocean:dev/gfdl Aug 30, 2019
@marshallward marshallward deleted the enable_codecov branch September 3, 2019 15:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants