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

added unittests with tox, pytest, and the geth dev rpc #30

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

peekg
Copy link

@peekg peekg commented Apr 26, 2017

No description provided.

peekg added 3 commits April 26, 2017 11:28
…leSerpent, and

eth_compileLLL calls
for more info: ethereum/EIPs#209
+ removed deprecated db_putString, db_getString, db_putHex, db_getHex
calls
+ bumped version
@emunsing
Copy link

emunsing commented Jul 6, 2017

As it stands, this fork does not fix the hex bug for call() and call with transactions() which means that the demo in REAMDE.md does not work.

I added a pull request to your peekg/ethjsonrpc fork which fixes this issue, and have also created a new pull request on the main Consensys branch.

Thanks for your big improvements over the previous version!

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.

2 participants