Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(blob): use default min gas price from core #2306

Merged
merged 2 commits into from
Jun 2, 2023
Merged

Conversation

distractedm1nd
Copy link
Collaborator

Gas estimation broke from an incorrect constant, that can now be replaced with an app import

@codecov-commenter
Copy link

Codecov Report

Merging #2306 (828bc54) into main (218ab99) will increase coverage by 0.22%.
The diff coverage is 20.00%.

@@            Coverage Diff             @@
##             main    #2306      +/-   ##
==========================================
+ Coverage   50.82%   51.04%   +0.22%     
==========================================
  Files         154      154              
  Lines        9671     9633      -38     
==========================================
+ Hits         4915     4917       +2     
+ Misses       4329     4291      -38     
+ Partials      427      425       -2     
Impacted Files Coverage Δ
api/gateway/state.go 0.00% <0.00%> (ø)
state/core_access.go 27.83% <ø> (ø)
blob/service.go 70.93% <25.00%> (ø)

... and 6 files with indirect coverage changes

@vgonkivs vgonkivs merged commit c6f4d68 into main Jun 2, 2023
@vgonkivs vgonkivs deleted the defaultgas-fromcore branch June 2, 2023 15:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:blob kind:fix Attached to bug-fixing PRs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants