Skip to content

Remove coverage #349

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

Merged
merged 1 commit into from
Jun 8, 2020
Merged

Remove coverage #349

merged 1 commit into from
Jun 8, 2020

Conversation

tiran
Copy link
Member

@tiran tiran commented Jun 6, 2020

At the moment coverage is causing more trouble than it's worth. Codecov
isn't giving meaningful results and we aren't using coverage to increase
test coverage.

Fixes: #348
Signed-off-by: Christian Heimes cheimes@redhat.com

@tiran tiran added the tests label Jun 6, 2020
At the moment coverage is causing more trouble than it's worth. Codecov
isn't giving meaningful results and we aren't using coverage to increase
test coverage.

Fixes: python-ldap#348
Signed-off-by: Christian Heimes <cheimes@redhat.com>
@encukou encukou merged commit c00694e into python-ldap:master Jun 8, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

coverage downgrade breaks RPM build on Fedora
2 participants