-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
Watch improvement #153
Watch improvement #153
Conversation
I fixed the tests, but I want to wait for travis && coveralls before merging. I don't know why, but coveralls is showing me, that coverage dropped to 50%. |
1 similar comment
it looks like bignumber.js is incorrectly included somewhere... but I can't find where :/ |
try { | ||
request.send(JSON.stringify(payload)); | ||
} catch(error) { | ||
throw errors.NoConnection(this.host); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it does not mean that it can't connect. This could be also incorrect response.
Conflicts: dist/ethereum-light.js.map dist/ethereum-light.min.js dist/ethereum.js.map dist/ethereum.min.js
I've merged 4 of the commits which were fine to master. |
1 similar comment
It looks like commit 86c941c is breaking coverage, but I don't know where exactly. |
add initial
getFilterLogs
check. add optional callback forwatch.get()
.One test marek wrote is failing @debris can you please fix it?