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

Test coverage c extension doc fix #593

Open
lrjball opened this issue Jun 16, 2020 · 0 comments
Open

Test coverage c extension doc fix #593

lrjball opened this issue Jun 16, 2020 · 0 comments
Labels
enhancement guide-edits Editing of doc or content needed topic-test

Comments

@lrjball
Copy link
Contributor

lrjball commented Jun 16, 2020

Currently in the test coverage section, specifically 5.2.1.4, it mentions building the C extension for coverage.py. For a while now coverage.py has come with the C extension by default, so this step is almost never required (see here: https://coverage.readthedocs.io/en/coverage-5.1/install.html#c-extension).
I'm thinking it would be worth mentioning in the guide that installing the C extension is only required if 'coverage --version' says 'Without C extension'.

@lrjball lrjball changed the title Test coverage doc fix Test coverage c extension doc fix Jun 16, 2020
@willingc willingc added enhancement guide-edits Editing of doc or content needed topic-test labels Oct 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement guide-edits Editing of doc or content needed topic-test
Projects
None yet
Development

No branches or pull requests

2 participants