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

Additional error for invalid gas #10327

Merged
merged 2 commits into from Feb 11, 2019
Merged

Additional error for invalid gas #10327

merged 2 commits into from Feb 11, 2019

Conversation

cheme
Copy link
Contributor

@cheme cheme commented Feb 11, 2019

This short PR return error on invalid gas. I also change a few operators for code homogeneity.

@cheme cheme added A0-pleasereview 🤓 Pull request needs code review. B1-patch-beta 🕷🕷 B0-patch-stable 🕷 Pull request should also be back-ported to the stable branch. labels Feb 11, 2019
@tomusdrw tomusdrw added A8-looksgood 🦄 Pull request is reviewed well. M4-core ⛓ Core client code / Rust. and removed A0-pleasereview 🤓 Pull request needs code review. labels Feb 11, 2019
@ordian ordian added this to the 2.4 milestone Feb 11, 2019
@kirushik kirushik merged commit d89b8d9 into master Feb 11, 2019
kirushik pushed a commit that referenced this pull request Feb 11, 2019
* Tag sensible place (ECHECH)

* Additional overflows checks.
kirushik pushed a commit that referenced this pull request Feb 11, 2019
* Tag sensible place (ECHECH)

* Additional overflows checks.
@5chdn 5chdn deleted the ec-return_errors branch February 12, 2019 09:52
5chdn pushed a commit that referenced this pull request Feb 12, 2019
* Tag sensible place (ECHECH)

* Additional overflows checks.
5chdn pushed a commit that referenced this pull request Feb 12, 2019
* Tag sensible place (ECHECH)

* Additional overflows checks.
ordian added a commit that referenced this pull request Apr 5, 2019
* master:
  no volumes are needed, just run -v volume:/path/in/the/container (#10345)
  Fixed misstype (#10351)
  snap: prefix version and populate candidate channel (#10343)
  Bundle protocol and packet_id together in chain sync (#10315)
  role back docker build image and docker deploy image to ubuntu:xenial based (#10338)
  change docker image based on debian instead of ubuntu due to the chan… (#10336)
  Don't add discovery initiators to the node table (#10305)
  fix(docker): fix not receives SIGINT (#10059)
  snap: official image / test (#10168)
  fix(add helper for timestamp overflows) (#10330)
  Additional error for invalid gas (#10327)
  Revive parity_setMinGasPrice RPC call (#10294)
  Add Statetest support for Constantinople Fix (#10323)
  fix(parity-clib): grumbles that were not addressed in #9920 (#10154)
  fix(light-rpc): Make `light_sync` generic (#10238)
  fix publish job (#10317)
  Secure WS-RPC: grant access to all apis (#10246)
  Make specification of protocol in SyncRequester::send_request explicit (#10295)
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. B0-patch-stable 🕷 Pull request should also be back-ported to the stable branch. M4-core ⛓ Core client code / Rust.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants