Skip to content

Conversation

@Pierre-Sassoulas
Copy link
Member

Description

One of the major pain point of developing pylint
for me was the 30s to launch even a single test
with tox and then not have coverage, so I think
this doc will be helpful for future contributors.

Type of Changes

Type
📜 Docs

@coveralls
Copy link

coveralls commented Aug 3, 2019

Coverage Status

Coverage remained the same at 90.004% when pulling 80794f3 on Pierre-Sassoulas:add-doc-on-fast-test-launching into 4bf6f03 on PyCQA:master.

@mattlbeck
Copy link
Contributor

mattlbeck commented Aug 4, 2019

It might be good to recommend cloning the astroid repo and installing it from master when running with pytest because. The astroid dep not being cutting-edge up-to-date caused me considerable confusion at first.

@Pierre-Sassoulas
Copy link
Member Author

@mattlbeck right, I had to open #3022 myself. I didn't know if cloning astroid was the best way to fix though. Seems like it is :).

@gyermolenko
Copy link
Contributor

+1 to show how to run single test module
Wouldn't it be nice to make some sections of readme same as in /doc/development_guide/contribute.rst ? (include parts of the document etc)

@Pierre-Sassoulas Pierre-Sassoulas force-pushed the add-doc-on-fast-test-launching branch 2 times, most recently from 120d8d0 to 5620e4f Compare August 9, 2019 17:26
@Pierre-Sassoulas
Copy link
Member Author

@gyermolenko I don't have a strong opinion about it, but we should do it in another merge request.

One of the major pain point of developping pylint
for me was the 30s to launch even a single test
with tox and then not have coverage.
I lost days of my life without this command. I mean, really.
@Pierre-Sassoulas Pierre-Sassoulas force-pushed the add-doc-on-fast-test-launching branch from 5620e4f to 80794f3 Compare August 16, 2019 16:00
@PCManticore PCManticore merged commit 380df00 into pylint-dev:master Aug 19, 2019
@Pierre-Sassoulas Pierre-Sassoulas deleted the add-doc-on-fast-test-launching branch August 19, 2019 16:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants