8000 Add web3_* RPC endpoints by aunyks · Pull Request #65 · ethereumjs/ethereumjs-client · GitHub
[go: up one dir, main page]

Skip to content
This repository was archived by the owner on Dec 10, 2020. It is now read-only.

Conversation

@aunyks
Copy link
Contributor
@aunyks aunyks commented Nov 8, 2018

This PR adds web3_clientVersion and web3_sha3 RPC endpoints.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.08%) to 95.775% when pulling 2b99ea9 on aunyks:master into 721fd68 on ethereumjs:master.

@vpulim
Copy link
Contributor
vpulim commented Nov 8, 2018

Thanks for this contribution! Can you please merge commits into a single commit that includes the tests so we don't pollute the commit history (maybe keep the comment as "Add web3_* rpc endpoints")? Will merge once that's done.

@aunyks
Copy link
Contributor Author
aunyks commented Nov 8, 2018

@vpulim If the PR is reviewed/approved, I believe Github gives me the option to squash the commits before I merge.

@vpulim
Copy link
Contributor
vpulim commented Nov 8, 2018

Ok, in that case feel free to merge

Copy link
Contributor
@vpulim vpulim left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@aunyks
Copy link
Contributor Author
aunyks commented Nov 8, 2018

@vpulim Ah, now Github's telling me that only those with write access can merge. Can you do it from your side? It should be available as a dropdown option on the merge button.

@vpulim vpulim merged commit c1be507 into ethereumjs:master Nov 8, 2018
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.

3 participants

0