Skip to content
This repository was archived by the owner on Apr 11, 2021. It is now read-only.

nitfixes: to get over the finish line#16

Merged
karlfloersch merged 6 commits intomasterfrom
12-13-close
Aug 27, 2020
Merged

nitfixes: to get over the finish line#16
karlfloersch merged 6 commits intomasterfrom
12-13-close

Conversation

@tynes
Copy link
Copy Markdown
Collaborator

@tynes tynes commented Aug 26, 2020

Description

This adds a fix and closes #12 and closes #13

Questions

Metadata

Fixes

  • Fixes # [Link to Issue]

Contributing Agreement

Copy link
Copy Markdown
Contributor

@karlfloersch karlfloersch left a comment

Choose a reason for hiding this comment

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

Time's up let's do this!

@karlfloersch karlfloersch merged commit 47da34c into master Aug 27, 2020
karlfloersch added a commit that referenced this pull request Nov 20, 2020
* Remove broken gas metering attempt

* Add newline to fix lint

* hardcode return value of eth_estimateGas to gas limit

* evm: fix log message

* state manager: safer calling

* rpc: leave comment to help future debugging

Co-authored-by: Karl Floersch <karl@karlfloersch.com>
Co-authored-by: Kevin Ho <kevinjho1996@gmail.com>
karlfloersch added a commit that referenced this pull request Nov 20, 2020
* Remove broken gas metering attempt

* Add newline to fix lint

* hardcode return value of eth_estimateGas to gas limit

* evm: fix log message

* state manager: safer calling

* rpc: leave comment to help future debugging

Co-authored-by: Karl Floersch <karl@karlfloersch.com>
Co-authored-by: Kevin Ho <kevinjho1996@gmail.com>
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