Skip to content

miner: gas limit strategy, target 3141592 & def gas price 50 Shannon#1578

Merged
obscuren merged 1 commit into
ethereum:developfrom
Gustav-Simonsson:frontier_thawing
Aug 4, 2015
Merged

miner: gas limit strategy, target 3141592 & def gas price 50 Shannon#1578
obscuren merged 1 commit into
ethereum:developfrom
Gustav-Simonsson:frontier_thawing

Conversation

@Gustav-Simonsson
Copy link
Copy Markdown

No description provided.

@robotally
Copy link
Copy Markdown

Vote Count Reviewers
👍 2 @fjl @obscuren
👎 0
Reaction Users
:shipit: @alexvandesande
:trollface: @obscuren

Updated: Tue Aug 4 15:12:43 UTC 2015

@obscuren
Copy link
Copy Markdown
Contributor

obscuren commented Aug 3, 2015

👍

@Gustav-Simonsson
Copy link
Copy Markdown
Author

@fjl @karalabe saw you guys discussed accounts test timeout before - know anything about the failing test?
@karalabe know anything about the failing test in the fetcher?

@obscuren obscuren added this to the 1.0.1 milestone Aug 3, 2015
@obscuren
Copy link
Copy Markdown
Contributor

obscuren commented Aug 3, 2015

@Gustav-Simonsson could you try to rebase and try again? I merged in a test that should get rid of one pesky failed test

@obscuren
Copy link
Copy Markdown
Contributor

obscuren commented Aug 4, 2015

Squash please

@zsfelfoldi
Copy link
Copy Markdown
Contributor

Default gpomax is 100 shannon here, I still think that should be raised. At least to 500 shannon imho.

@Gustav-Simonsson Gustav-Simonsson force-pushed the frontier_thawing branch 2 times, most recently from 397dc69 to d9c2254 Compare August 4, 2015 13:16
@alexvandesande
Copy link
Copy Markdown

:shipit:

@fjl
Copy link
Copy Markdown
Contributor

fjl commented Aug 4, 2015

👍

obscuren added a commit that referenced this pull request Aug 4, 2015
miner: gas limit strategy, target 3141592 & def gas price 50 Shannon
@obscuren obscuren merged commit 56219a5 into ethereum:develop Aug 4, 2015
@obscuren obscuren removed the review label Aug 4, 2015
maoueh pushed a commit to streamingfast/go-ethereum that referenced this pull request May 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants