Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Use MockedHost from EVMC 7.1.0 #212

Merged
merged 2 commits into from
Nov 29, 2019
Merged

Use MockedHost from EVMC 7.1.0 #212

merged 2 commits into from
Nov 29, 2019

Conversation

chfast
Copy link
Member

@chfast chfast commented Nov 21, 2019

  • Upgrade EVMC to 7.1.0.
  • Use the MockedHost from EVMC (previously copied from here).

@chfast chfast requested review from axic and gumb0 November 21, 2019 17:06
@codecov-io
Copy link

codecov-io commented Nov 21, 2019

Codecov Report

Merging #212 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #212   +/-   ##
=======================================
  Coverage   99.88%   99.88%           
=======================================
  Files           5        5           
  Lines         851      851           
  Branches      113      113           
=======================================
  Hits          850      850           
  Misses          1        1

@chfast chfast force-pushed the mocked_host branch 2 times, most recently from 9718cba to ac7cb1b Compare November 29, 2019 10:01
@axic
Copy link
Member

axic commented Nov 29, 2019

I guess you want to wait for the evmc7.1 release before this is merged?

@chfast chfast changed the title Use MockedHost from EVMC Use MockedHost from EVMC 7.1.0 Nov 29, 2019
@chfast chfast merged commit a960646 into master Nov 29, 2019
@chfast chfast deleted the mocked_host branch November 29, 2019 14:45
jwasinger pushed a commit to jwasinger/evmone that referenced this pull request Apr 27, 2021
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants