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

Miner's gas price gets updated dynamically #1570

Merged
merged 8 commits into from Jul 9, 2016
Merged

Conversation

gavofyork
Copy link
Contributor

No description provided.

@gavofyork gavofyork added the A3-inprogress ⏳ Pull request is in progress. No review needed at this stage. label Jul 8, 2016
@gavofyork gavofyork added A0-pleasereview 🤓 Pull request needs code review. and removed A3-inprogress ⏳ Pull request is in progress. No review needed at this stage. labels Jul 8, 2016
let client = try!(Client::new().map_err(|_| ()));
trace!(target: "miner", "Starting price info request...");
thread::spawn(move || {
trace!(target: "miner", "Inside thread...");
Copy link
Collaborator

Choose a reason for hiding this comment

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

Should be reworded or removed

@arkpar arkpar added A6-mustntgrumble 💦 Pull request has areas for improvement. The author need not address them before merging. and removed A0-pleasereview 🤓 Pull request needs code review. labels Jul 9, 2016
@arkpar arkpar added A8-looksgood 🦄 Pull request is reviewed well. and removed A6-mustntgrumble 💦 Pull request has areas for improvement. The author need not address them before merging. labels Jul 9, 2016
@NikVolf NikVolf merged commit ae757af into master Jul 9, 2016
@arkpar arkpar deleted the dynamic-gas-price branch October 3, 2016 15:36
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.

None yet

3 participants