Skip to content

change targetgaslimit from 50M to 80M for devnet#247

Open
gzliudan wants to merge 1 commit intoXinFinOrg:masterfrom
gzliudan:high-gas-limit
Open

change targetgaslimit from 50M to 80M for devnet#247
gzliudan wants to merge 1 commit intoXinFinOrg:masterfrom
gzliudan:high-gas-limit

Conversation

@gzliudan
Copy link
Contributor

@gzliudan gzliudan commented Mar 5, 2026

Summary by CodeRabbit

  • Chores
    • Increased the maximum gas limit for transactions in the development environment from 50,000,000 to 80,000,000.

@coderabbitai
Copy link

coderabbitai bot commented Mar 5, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 1a4db114-2cc8-4f0e-9be0-aa02df4af461

📥 Commits

Reviewing files that changed from the base of the PR and between 2714982 and 3905326.

📒 Files selected for processing (1)
  • devnet/start.sh

📝 Walkthrough

Walkthrough

The devnet startup script's Ethereum/XDC target gas limit parameter is increased from 50,000,000 to 80,000,000 gas units, adjusting the maximum gas limit configuration for transactions in the development environment.

Changes

Cohort / File(s) Summary
Gas Limit Configuration
devnet/start.sh
Updated Ethereum/XDC target gas limit CLI argument from 50000000 to 80000000.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Suggested reviewers

  • AnilChinchawale

Poem

🐰 Hopping through the devnet with glee,
Gas limits raised from fifty to eighty we see,
More transactions can flow, no need to delay,
The blockchain now bounces with joy every day!

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely summarizes the main change: updating the target gas limit parameter in devnet from 50M to 80M.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

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