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

Add configuration to exclude tests and empty files from Coveralls.io report #274

Merged
merged 1 commit into from
Dec 13, 2015

Conversation

micbou
Copy link
Collaborator

@micbou micbou commented Dec 13, 2015

There is no point to get the coverage of the tests and __init__.py files are empties so we exclude them from the coverage report.

Review on Reviewable

Exclude tests folder and __init__.py empty files from Coveralls.io
report.
@puremourning
Copy link
Member

:lgtm:


Review status: 0 of 1 files reviewed at latest revision, all discussions resolved.


Comments from the review on Reviewable.io

@puremourning
Copy link
Member

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.


Comments from the review on Reviewable.io

@vheon
Copy link
Contributor

vheon commented Dec 13, 2015

:lgtm: @homu r+


Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved, some commit checks failed.


Comments from the review on Reviewable.io

@homu
Copy link
Contributor

homu commented Dec 13, 2015

📌 Commit 4211ec3 has been approved by vheon

@homu
Copy link
Contributor

homu commented Dec 13, 2015

⚡ Test exempted - status

@homu homu merged commit 4211ec3 into ycm-core:master Dec 13, 2015
homu added a commit that referenced this pull request Dec 13, 2015
Add configuration to exclude tests and empty files from Coveralls.io report

There is no point to get the coverage of the tests and `__init__.py` files are empties so we exclude them from the coverage report.

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/valloric/ycmd/274)
<!-- Reviewable:end -->
@micbou micbou deleted the coverage-exclude branch December 15, 2015 11:15
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.

None yet

4 participants