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.

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

Remove coveralls #236

Closed
vesln opened this issue Jan 25, 2014 · 2 comments · Fixed by #237
Closed

Remove coveralls #236

vesln opened this issue Jan 25, 2014 · 2 comments · Fixed by #237

Comments

@vesln
Copy link
Member

vesln commented Jan 25, 2014

Coveralls is a nice but it's more than unstable. It sucks to have failing builds because of their issues. I have tried to speak with them a couple of times about this problem (it's not only in chai it's across all my projects) but no answer so far. I say we remove it for now and if they happen to fix their problems in the future, we add it back.
@logicalparadox
Copy link
Member

Yeah, thats fine. How about istanbul cli reports in the meantime?

@vesln
Copy link
Member Author

vesln commented Jan 26, 2014

ack!

@vesln vesln mentioned this issue Jan 26, 2014
Merged
@meeber meeber mentioned this issue Feb 3, 2017
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Remove coveralls & generate coverage with istanbul chaijs/chai
2 participants
@logicalparadox @vesln