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

docs: update the simplified make alltests instructions to include reference to installing coverage #1109

Merged

Conversation

kevinmarsh
Copy link
Contributor

The "Alternative unit tests with pytest" instructions in CONTRIBUTING.md no longer work after switching from nose -> pytest. You need to also install coverage and pytest-cov in order to run make alltests

ERROR: usage: pytest [options] [file_or_dir] [file_or_dir] [...]                                                                                           
pytest: error: unrecognized arguments: --cov=tqdm --cov-fail-under=80                                                                                      

@casperdcl casperdcl changed the base branch from master to devel January 9, 2021 17:08
@casperdcl casperdcl merged commit 4ccc5e8 into tqdm:devel Jan 9, 2021
@casperdcl casperdcl added question/docs ‽ Documentation clarification candidate to-merge ↰ Imminent labels Jan 9, 2021
@casperdcl casperdcl added this to Done in Casper Jan 9, 2021
@casperdcl casperdcl self-assigned this Jan 9, 2021
@casperdcl casperdcl mentioned this pull request Jan 9, 2021
@casperdcl casperdcl added this to the Non-breaking milestone Jan 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question/docs ‽ Documentation clarification candidate to-merge ↰ Imminent
Projects
Casper
  
Done
Development

Successfully merging this pull request may close these issues.

None yet

2 participants