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

Include test-helpers from ethjson #11045

Merged
merged 1 commit into from
Sep 13, 2019
Merged

Include test-helpers from ethjson #11045

merged 1 commit into from
Sep 13, 2019

Conversation

dvdplm
Copy link
Collaborator

@dvdplm dvdplm commented Sep 13, 2019

Closes #11044

@dvdplm dvdplm self-assigned this Sep 13, 2019
@dvdplm dvdplm added A0-pleasereview 🤓 Pull request needs code review. A2-insubstantial 👶 Pull request requires no code review (e.g., a sub-repository hash update). labels Sep 13, 2019
@ordian ordian added this to the 2.7 milestone Sep 13, 2019
@ordian ordian added A8-looksgood 🦄 Pull request is reviewed well. and removed A0-pleasereview 🤓 Pull request needs code review. labels Sep 13, 2019
@ordian
Copy link
Collaborator

ordian commented Sep 13, 2019

Can we add a check to CI?

Copy link
Collaborator

@niklasad1 niklasad1 left a comment

Choose a reason for hiding this comment

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

nice, sorry missed that in my PR

@niklasad1 niklasad1 merged commit 0bd2348 into master Sep 13, 2019
@niklasad1 niklasad1 deleted the dp/fix/compile-evmbin branch September 13, 2019 07:38
dvdplm added a commit that referenced this pull request Sep 13, 2019
* master: (70 commits)
  ethcore: remove `test-helper feat` from build (#11047)
  Include test-helpers from ethjson (#11045)
  [ethcore]: cleanup dependencies (#11043)
  add more tx tests (#11038)
  Fix parallel transactions race-condition (#10995)
  [ethcore]: make it compile without `test-helpers` feature (#11036)
  Benchmarks for block verification (#11035)
  Move snapshot related traits to their proper place (#11012)
  cleanup json crate (#11027)
  [spec] add istanbul test spec (#11033)
  [json-spec] make blake2 pricing spec more readable (#11034)
  Add blake2_f precompile (#11017)
  Add new line after writing block to hex file. (#10984)
  fix: remove unused error-chain (#11028)
  fix: remove needless use of itertools (#11029)
  Convert `std::test` benchmarks to use Criterion (#10999)
  Fix block detail updating (#11015)
  [trace] introduce trace failed to Ext (#11019)
  cli: update usage and version headers (#10924)
  [private-tx] remove unused rand (#11024)
  ...
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A2-insubstantial 👶 Pull request requires no code review (e.g., a sub-repository hash update). A8-looksgood 🦄 Pull request is reviewed well.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Cannot build package evmbin
3 participants