-
Notifications
You must be signed in to change notification settings - Fork 5
Miscellaneous changes #100
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
Conversation
willrogers
left a comment
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.
Generally looks good, I've made a few comments that you can respond to.
| $ python -m pytest | ||
|
|
||
| To see a coverage report, check pep8 and pyflakes:: | ||
| To see a coverage report, check pytest-cov:: |
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.
How about:
To see a coverage report and style violations, you can use pytest-cov, pep8 and pyflakes:
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.
I thought that we use flake8 instead of pep8 and pyflakes.
So perhaps:
To see a coverage report, check pytest-cov:
$ python -m pytest --cov-report term-missing --cov=pytac
To see style violations, use flake8:
$ flake8
Or we could use this plugin:
To see a coverage report and style violations, you can use pytest-cov and flake8:
$ python -m pytest --cov-report term-missing --cov=pytac --flake8
This is pretty much everything that was left on my todo list for pytac.
If you remember we discovered that our errors were being chained rather than replaced, when using Python 3, e.g.
According to pep3134 this is the intended behaviour now; pep415 gives ways to avoid this, but they are not compatible with python 2.7 as far as I can tell. So for it seems like we are stuck with this behaviour.