Simplify test running directions - #552
Conversation
|
|
||
| To run the Travis CI tests, you'll have to create a build the pycodestyle | ||
| package using tox:: | ||
| To run the tests, use tox:: |
There was a problem hiding this comment.
What about "To run the tests the same way Travis CI runs them, use tox::"
I think that captures the intent of the original awkward phrasing.
There was a problem hiding this comment.
To me, the intent is to "run tests".
Instead of running tests "the way Travis CI does", CI should be running the tests like we do.
There was a problem hiding this comment.
The core developers use tox. Not everyone will want to. That's the point of specifying that you're running them the way the CI runs them - if they get incongruous results, they can replicate that.
|
You closed this by accident, yes? |
|
No, it was on purpose. Since you disagree with my fundamental reasoning on why I made the change to remove "Travis CI" and I presented all my arguments already, I don't see a future for this PR. The change is so trivial, its's not worth spending more time on it, which is why I withdrew it. Any further changes to the wording are at your discretion. |
3 participants
The wording was poor, imo.