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

Migrate to pytest and implement code coverage. #167

Merged
merged 29 commits into from
Aug 21, 2020
Merged

Migrate to pytest and implement code coverage. #167

merged 29 commits into from
Aug 21, 2020

Conversation

sdhutchins
Copy link
Member

@sdhutchins sdhutchins commented Aug 21, 2020

Fixes issue #25

Changes proposed in this pull request:

  • Migrate to pytest from nose given nose is no longer actively maintained.
  • Added pytest.ini
  • Implemented code coverage in .travis.yml
  • Deleted unneeded mpi tests.
  • Fixed syntax warnings.

@codecov
Copy link

codecov bot commented Aug 21, 2020

Codecov Report

❗ No coverage uploaded for pull request base (dev-master@93bb265). Click here to learn what that means.
The diff coverage is n/a.

Impacted file tree graph

@@              Coverage Diff              @@
##             dev-master     #167   +/-   ##
=============================================
  Coverage              ?   15.13%           
=============================================
  Files                 ?       67           
  Lines                 ?     4182           
  Branches              ?        0           
=============================================
  Hits                  ?      633           
  Misses                ?     3549           
  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 93bb265...3305c50. Read the comment docs.

@sdhutchins sdhutchins changed the title Implement code coverage. Migrate to pytest and implement code coverage. Aug 21, 2020
@grabear grabear merged commit 1c1bf90 into dev-master Aug 21, 2020
@sdhutchins sdhutchins deleted the codecov branch August 21, 2020 15:48
@sdhutchins sdhutchins added 👁️ Status: Review Needed 👁️ 👍 Status: Accepted 👍 dependencies Pull requests that update a dependency file and removed 👁️ Status: Review Needed 👁️ dependencies Pull requests that update a dependency file labels Aug 21, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants