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.

Sign up for GitHub

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

Jump to bottom

gitignore coverage file #40

Closed

benoitbryon wants to merge 1 commit into master from 38-coverage

Contributor

benoitbryon commented Jan 13, 2015

Refs #38


          Refs #38 - gitignore coverage file.

Contributor Author

benoitbryon commented Jan 13, 2015

See 595d301#commitcomment-9215461

I think there should not be a coverage file when you commit.
Because there is a coverage erase in the tox configuration file: https://github.com/novapost/pydocusign/blob/595d30183ae68d2589e7ecd9a5ebfcedd95d7c10/tox.ini#L15
If you have a .coverage, it means that the tests did not pass.

At the moment, it means "wontfix".
Explain the issue with more details if you want it merged.

bersace closed this

bersace deleted the 38-coverage branch

January 23, 2015 12:43

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet