-
Notifications
You must be signed in to change notification settings - Fork 4
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
Change to pytest #111
Change to pytest #111
Conversation
It could be its own PR, but if you're updating dependencies, downgrading ASE (#112) might be sensible here. |
.github/workflows/ci.yml
Outdated
poetry run pytest --cov=abcd --cov-report xml --cov-report term:skip-covered | ||
|
||
- name: Upload coverage reports to Codecov | ||
uses: codecov/[email protected] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Any reason for pinning to 4.0.1 specifically? I think the most recent release is 4.4.1.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
no special intention, the codecov instructions provided this snippet
Welcome to Codecov 🎉Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests. Thanks for integrating Codecov - We've got you covered ☂️ |
for #110
unittest
testing suite intopytest
pytest
& run coveragen.b. Codecov token is needed in the repo.