Skip to content

feat: revert if task required gas limit is close to the fusaka tx gas cap#1277

Merged
mds1 merged 6 commits into
mainfrom
feat/check-gas-limit
Dec 22, 2025
Merged

feat: revert if task required gas limit is close to the fusaka tx gas cap#1277
mds1 merged 6 commits into
mainfrom
feat/check-gas-limit

Conversation

@mds1
Copy link
Copy Markdown
Contributor

@mds1 mds1 commented Oct 30, 2025

No description provided.

@mds1 mds1 requested review from a team as code owners October 30, 2025 18:22
@mds1 mds1 requested a review from ajsutton October 30, 2025 18:22
Comment thread src/tasks/MultisigTask.sol Outdated
mds1 and others added 2 commits October 30, 2025 11:24
Co-authored-by: graphite-app[bot] <96075541+graphite-app[bot]@users.noreply.github.com>
JosepBove
JosepBove previously approved these changes Oct 30, 2025
Copy link
Copy Markdown
Member

@JosepBove JosepBove left a comment

Choose a reason for hiding this comment

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

Looks good!

alcueca
alcueca previously approved these changes Oct 31, 2025
Copy link
Copy Markdown
Contributor

@alcueca alcueca left a comment

Choose a reason for hiding this comment

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

LGTM

Comment thread src/tasks/MultisigTask.sol Outdated
@mds1
Copy link
Copy Markdown
Contributor Author

mds1 commented Oct 31, 2025

We will hold off on merging this until after Jovian since this limit doesn't apply to mainnet yet

@mds1 mds1 mentioned this pull request Oct 31, 2025
@ajsutton ajsutton removed their request for review November 14, 2025 02:47
@mds1 mds1 enabled auto-merge December 22, 2025 18:44
Comment thread src/ci.just
Comment thread src/tasks/MultisigTask.sol
@mds1 mds1 added this pull request to the merge queue Dec 22, 2025
Merged via the queue into main with commit 4613fdd Dec 22, 2025
21 checks passed
@mds1 mds1 deleted the feat/check-gas-limit branch December 22, 2025 20:09
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.

5 participants