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

fixed outdated pytest tardis command in documentation #1876

Merged
merged 1 commit into from
Jan 25, 2022

Conversation

afloers
Copy link
Contributor

@afloers afloers commented Jan 25, 2022

The documentation currently contains the old python setup.py test command.
This PR replaces it with the new pytest tardis command.

How has this been tested?

  • Testing pipeline.
  • Other.

Examples

Type of change

  • Bug fix.
  • New feature.
  • Breaking change.
  • None of the above.

Checklist

  • My change requires a change to the documentation.
    • I have updated the documentation accordingly.
    • (optional) I have built the documentation on my fork following the instructions.
  • I have assigned and requested two reviewers for this pull request.

@codecov
Copy link

codecov bot commented Jan 25, 2022

Codecov Report

Merging #1876 (4dc3fb2) into master (c44183d) will not change coverage.
The diff coverage is n/a.

❗ Current head 4dc3fb2 differs from pull request most recent head 7e2ca39. Consider uploading reports for the commit 7e2ca39 to get more accurate results
Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1876   +/-   ##
=======================================
  Coverage   62.24%   62.24%           
=======================================
  Files          67       67           
  Lines        6830     6830           
=======================================
  Hits         4251     4251           
  Misses       2579     2579           

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 c44183d...7e2ca39. Read the comment docs.

@wkerzendorf wkerzendorf merged commit 0849617 into tardis-sn:master Jan 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

4 participants