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

Fix ethcore/benches build.#10964

Merged
ordian merged 1 commit into
openethereum:masterfrom
poanetwork:afck-fix-bench-build
Aug 13, 2019
Merged

Fix ethcore/benches build.#10964
ordian merged 1 commit into
openethereum:masterfrom
poanetwork:afck-fix-bench-build

Conversation

@afck
Copy link
Copy Markdown
Contributor

@afck afck commented Aug 13, 2019

No description provided.

@parity-cla-bot
Copy link
Copy Markdown

It looks like @afck signed our Contributor License Agreement. 👍

Many thanks,

Parity Technologies CLA Bot

@afck
Copy link
Copy Markdown
Contributor Author

afck commented Aug 13, 2019

I think this was overlooked in #10949. It would probaby make sense to add something like cargo check --benches to CI.

Copy link
Copy Markdown
Member

@ordian ordian left a comment

Choose a reason for hiding this comment

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

Nice catch, thank you!

@ordian ordian added the A8-looksgood 🦄 Pull request is reviewed well. label Aug 13, 2019
@ordian ordian added this to the 2.7 milestone Aug 13, 2019
Copy link
Copy Markdown
Collaborator

@dvdplm dvdplm left a comment

Choose a reason for hiding this comment

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

Lovely, I missed checking the benchmarks.

@ordian ordian merged commit a23f5b8 into openethereum:master Aug 13, 2019
@afck afck deleted the afck-fix-bench-build branch August 13, 2019 13:29
ordian added a commit that referenced this pull request Aug 15, 2019
* master:
  [evmbin] fix compilation (#10976)
  Update to latest trie version. (#10972)
  [blooms-db] Fix benchmarks (#10974)
  Fix ethcore/benches build. (#10964)
  tx-pool: accept local tx with higher gas price when pool full (#10901)
  Disable unsyncable expanse chain (#10926)
  Extract Machine from ethcore (#10949)
  removed redundant state_root function from spec, improve spec error types (#10955)
  Add support for Energy Web Foundation's new chains (#10957)
  [evmbin] add more tests to main.rs (#10956)
  Fix compiler warnings in util/io and upgrade to edition 2018 Upgrade mio to latest (#10953)
  unify loading spec && further spec cleanups (#10948)
  refactor: Refactor evmbin CLI (#10742)
  journaldb changes (#10929)
  Allow default block parameter to be blockHash (#10932)
  Enable sealing when engine is ready (#10938)
  Fix some warnings and typos. (#10941)
  Updated security@parity.io key (#10939)
  Change the return type of step_inner function. (#10940)
ordian added a commit that referenced this pull request Aug 15, 2019
* master:
  Verify transaction against its block during import (#10954)
  [evmbin] fix compilation (#10976)
  Update to latest trie version. (#10972)
  [blooms-db] Fix benchmarks (#10974)
  Fix ethcore/benches build. (#10964)
  tx-pool: accept local tx with higher gas price when pool full (#10901)
  Disable unsyncable expanse chain (#10926)
dvdplm added a commit that referenced this pull request Aug 15, 2019
* master:
  Extract the Engine trait (#10958)
  Better error message for rpc gas price errors (#10931)
  [.gitlab.yml] cargo check ethcore benches (#10965)
  Verify transaction against its block during import (#10954)
  [evmbin] fix compilation (#10976)
  Update to latest trie version. (#10972)
  [blooms-db] Fix benchmarks (#10974)
  Fix ethcore/benches build. (#10964)
  tx-pool: accept local tx with higher gas price when pool full (#10901)
  Disable unsyncable expanse chain (#10926)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

A8-looksgood 🦄 Pull request is reviewed well.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants