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

Max gas limit and min gas price #4661

Merged
merged 3 commits into from
Feb 24, 2017
Merged

Max gas limit and min gas price #4661

merged 3 commits into from
Feb 24, 2017

Conversation

arkpar
Copy link
Collaborator

@arkpar arkpar commented Feb 24, 2017

No description provided.

@arkpar arkpar added A3-inprogress ⏳ Pull request is in progress. No review needed at this stage. M4-core ⛓ Core client code / Rust. A0-pleasereview 🤓 Pull request needs code review. and removed A3-inprogress ⏳ Pull request is in progress. No review needed at this stage. labels Feb 24, 2017
@gavofyork
Copy link
Contributor

might be sensible to force gas ceil target to be at most the max gas limit, otherwise a misconfigured miner could start creating blocks that it can't import.

@arkpar arkpar added A3-inprogress ⏳ Pull request is in progress. No review needed at this stage. and removed A0-pleasereview 🤓 Pull request needs code review. labels Feb 24, 2017
@arkpar arkpar added A0-pleasereview 🤓 Pull request needs code review. and removed A3-inprogress ⏳ Pull request is in progress. No review needed at this stage. labels Feb 24, 2017
@gavofyork gavofyork added A8-looksgood 🦄 Pull request is reviewed well. and removed A0-pleasereview 🤓 Pull request needs code review. labels Feb 24, 2017
@gavofyork gavofyork merged commit ef7998f into master Feb 24, 2017
@gavofyork gavofyork deleted the gas-limits branch February 24, 2017 16:34
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. M4-core ⛓ Core client code / Rust.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants