Skip to content
This repository has been archived by the owner on Feb 26, 2024. It is now read-only.

feat: add ability to use blockHash with eth_getLogs #635

Merged

Conversation

tynes
Copy link
Contributor

@tynes tynes commented Sep 29, 2020

Implements the ability to use blockHash with eth_getLogs. This closes #136

See EIP 234

@coveralls
Copy link

Coverage Status

Coverage increased (+0.05%) to 82.749% when pulling b5c2115 on ethereum-optimism:eth_getLogs-blockhash into 53f653e on trufflesuite:develop.

@coveralls
Copy link

coveralls commented Sep 29, 2020

Coverage Status

Coverage increased (+0.05%) to 82.749% when pulling 63848fc on ethereum-optimism:eth_getLogs-blockhash into 53f653e on trufflesuite:develop.

@tynes
Copy link
Contributor Author

tynes commented Sep 29, 2020

Any feedback on this PR or ideas on when it can be merged? In the meantime, we published it at @eth-optimism/ganache-core for usage in our testing. We are publishing a docker image ethereumoptimism/ganache-cli on Dockerhub that uses the eth-optimism fork

@davidmurdoch davidmurdoch changed the title rpc: add ability to use blockHash with eth_getLogs feat: add ability to use blockHash with eth_getLogs Sep 29, 2020
@davidmurdoch davidmurdoch merged commit fb16896 into trufflesuite:develop Sep 29, 2020
davidmurdoch added a commit that referenced this pull request Sep 29, 2020
davidmurdoch added a commit that referenced this pull request Sep 29, 2020
@davidmurdoch
Copy link
Member

davidmurdoch commented Sep 29, 2020

I didn't notice you changed the package name since I last looked at it and accidentally merged it with that extra commit. Reverted.

@tynes
Copy link
Contributor Author

tynes commented Sep 30, 2020

Sorry about that extra commit, I didn't mean to push that here. Will reopen without that commit. Meant to push that to my local fork.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Implement EIP-234
3 participants