Skip to content

Update contract sizes benchmark#1665

Merged
bidzyyys merged 1 commit intomainfrom
update/contract-sizes-23796841385
Mar 31, 2026
Merged

Update contract sizes benchmark#1665
bidzyyys merged 1 commit intomainfrom
update/contract-sizes-23796841385

Conversation

@github-actions
Copy link
Copy Markdown
Contributor

@github-actions github-actions Bot commented Mar 31, 2026

This PR updates the contract size benchmarks after a recent merge to main.

Summary by CodeRabbit

  • Chores
    • Updated contract size benchmarks with metrics for new test contracts.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Mar 31, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: cc405753-8e00-4225-a5c6-a69ebdca7ef9

📥 Commits

Reviewing files that changed from the base of the PR and between 5452a72 and 19bfe69.

📒 Files selected for processing (1)
  • benches/contract_sizes.json

Walkthrough

Added benchmark contract size measurements to benches/contract_sizes.json for three new OpenZeppelin contracts: ERC20FlashMintConfiguredMock, ERC20FlashMintMock, and ERC3156FlashBorrowerMock. The entries include compiled bytecode and contract class sizes.

Changes

Cohort / File(s) Summary
Benchmark Data
benches/contract_sizes.json
Added contract size metrics for three flash mint-related contracts under both bytecode and contract_class mappings, documenting compiled sizes in felts and bytes respectively.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Suggested reviewers

  • ericnordelo
  • immrsd
  • bidzyyys

Poem

🐰 Three contracts now measured and weighed,
Flash mint magic in metrics displayed,
Bytecode and class sizes all set,
The benchmark collection grows yet,
OpenZeppelin's benchmarks cascade! ✨

🚥 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 'Update contract sizes benchmark' directly and accurately reflects the main change in the pull request, which updates contract size benchmark entries in the benches/contract_sizes.json file.
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
  • Commit unit tests in branch update/contract-sizes-23796841385

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

@ericnordelo ericnordelo requested a review from bidzyyys March 31, 2026 14:00
@bidzyyys bidzyyys merged commit 10c9d10 into main Mar 31, 2026
7 checks passed
@bidzyyys bidzyyys deleted the update/contract-sizes-23796841385 branch March 31, 2026 14:35
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.

2 participants