Skip to content

config: set the L2 gaslimit to 30 million in config#5194

Merged
tynes merged 1 commit intodevelopfrom
fix/gaslimit-config
Mar 22, 2023
Merged

config: set the L2 gaslimit to 30 million in config#5194
tynes merged 1 commit intodevelopfrom
fix/gaslimit-config

Conversation

@tynes
Copy link
Contributor

@tynes tynes commented Mar 18, 2023

Description

Ensures that things are smooth once #5112 is merged. A too small of gas limit will break things after that PR is merged. We will run with 30 million limit on mainnet, so we should ensure that our devnet config reflects that.

Ensures that things are smooth once #5112
is merged. A too small of gas limit will break things after
that PR is merged. We will run with 30 million limit on mainnet,
so we should ensure that our devnet config reflects that.
@tynes tynes requested a review from a team as a code owner March 18, 2023 00:07
@tynes tynes requested a review from smartcontracts March 18, 2023 00:07
@changeset-bot
Copy link

changeset-bot bot commented Mar 18, 2023

⚠️ No Changeset found

Latest commit: 4a0617a

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@netlify
Copy link

netlify bot commented Mar 18, 2023

Deploy Preview for opstack-docs canceled.

Name Link
🔨 Latest commit 4a0617a
🔍 Latest deploy log https://app.netlify.com/sites/opstack-docs/deploys/641500acb0371900073db52f

@codecov
Copy link

codecov bot commented Mar 18, 2023

Codecov Report

Merging #5194 (4a0617a) into develop (346ac69) will decrease coverage by 0.57%.
The diff coverage is n/a.

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff             @@
##           develop    #5194      +/-   ##
===========================================
- Coverage    41.92%   41.35%   -0.57%     
===========================================
  Files          354      333      -21     
  Lines        21977    21348     -629     
  Branches       776      661     -115     
===========================================
- Hits          9213     8829     -384     
+ Misses       12063    11815     -248     
- Partials       701      704       +3     
Flag Coverage Δ
bedrock-go-tests 37.96% <ø> (-0.03%) ⬇️
contracts-bedrock-tests 51.04% <ø> (ø)
contracts-tests 98.86% <ø> (ø)
core-utils-tests ?
dtl-tests 47.15% <ø> (ø)
fault-detector-tests 33.88% <ø> (ø)
sdk-tests 39.05% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

see 23 files with indirect coverage changes

@tynes tynes merged commit aeda9d9 into develop Mar 22, 2023
@tynes tynes deleted the fix/gaslimit-config branch March 22, 2023 15:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant