-
-
Notifications
You must be signed in to change notification settings - Fork 186
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
Fix unhashable exception #230
Fix unhashable exception #230
Conversation
@rodrigc appreciate the quick fix, thanks! |
@TheKevJames Thanks for merging the fix. I can confirm that it works. In the Twisted project, I am occasionally seeing errors, such as the one at this link: https://travis-ci.com/github/twisted/twisted/jobs/361796000 that look like:
Is there a problem on the coveralls server? |
I also saw this error at https://travis-ci.com/github/twisted/twisted/jobs/361796000
|
Unfortunately, it seems the Coveralls API just isn't all that reliable -- we've gotten a bunch of reports like this (and other coveralls libraries have seen the same sort of thing) and they mostly just amount to intermittent API issues. We're not associated with Coveralls at all, so there's nothing I can do there. The 422 error might be something on our side -- if you can reproduce it within some specific set of inputs, it might be something we can avoid! Feel free to open up an issue if so, I'd be happy to take a look. |
Thanks for the response. Yes this was an intermittent problem and I have not hit it lately. |
Fixes #229