Skip to content

Bump snforge to 0.51.2 #1563

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.

Sign up for GitHub

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

Jump to bottom
Merged
immrsd merged 2 commits into from
Nov 3, 2025
Merged

Bump snforge to 0.51.2 #1563

immrsd merged 2 commits into from
Nov 3, 2025

Conversation

@immrsd
Copy link
Collaborator

@immrsd immrsd commented Nov 1, 2025

No description provided.

@immrsd immrsd requested a review from ericnordelo November 1, 2025 12:27
@immrsd immrsd self-assigned this Nov 1, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Nov 1, 2025 •
edited
Loading

🧪 Cairo Contract Size Benchmark Diff

BYTECODE SIZE (felts) (limit: 81,920 felts)

No changes in felts.

SIERRA CONTRACT CLASS SIZE (bytes) (limit: 4,089,446 bytes)

No changes in bytes.

This comment was generated automatically from benchmark diffs.

@codecov
Copy link

codecov bot commented Nov 1, 2025 •
edited
Loading

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 92.39%. Comparing base (0ffd66f) to head (9a5245f).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files 
@@           Coverage Diff           @@
##             main    #1563   +/-   ##
=======================================
  Coverage   92.39%   92.39%           
=======================================
  Files          85       85           
  Lines        2275     2275           
=======================================
  Hits         2102     2102           
  Misses        173      173

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0ffd66f...9a5245f. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

ericnordelo
ericnordelo approved these changes Nov 3, 2025
View reviewed changes
Copy link
Member

@ericnordelo ericnordelo left a comment

Choose a reason for hiding this comment

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

LGTM!

@immrsd immrsd merged commit 7f2d90b into main Nov 3, 2025
10 checks passed
ericnordelo added a commit that referenced this pull request Nov 20, 2025
@ericnordelo @dependabot
@github-actions @vastonus @immrsd
Improve AccessControlDefaultAdminRules admin transfer delay change …
321064b 
…logic (#1567)

* Chore(deps): Bump stefanzweifel/git-auto-commit-action (#1553)

Bumps [stefanzweifel/git-auto-commit-action](https://github.com/stefanzweifel/git-auto-commit-action) from 6.0.1 to 7.0.0.
- [Release notes](https://github.com/stefanzweifel/git-auto-commit-action/releases)
- [Changelog](https://github.com/stefanzweifel/git-auto-commit-action/blob/master/CHANGELOG.md)
- [Commits](stefanzweifel/git-auto-commit-action@778341a...28e16e8)

---
updated-dependencies:
- dependency-name: stefanzweifel/git-auto-commit-action
  dependency-version: 7.0.0
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Update foundry to 0.51.0 (#1555)

* feat: update to 0.51.0

* feat: update CHANGELOG

* Update packages/testing/src/common.cairo

Co-authored-by: Copilot <[email protected]>

* feat: update to 0.51.1

---------

Co-authored-by: Copilot <[email protected]>

* Release openzeppelin_testing v6.0.0 (#1556)

* feat: update to 0.51.0

* feat: update CHANGELOG

* Update packages/testing/src/common.cairo

Co-authored-by: Copilot <[email protected]>

* feat: update CHANGELOG

* Bump openzeppelin_testing version to 6.0.0 and update docs

* feat: update to 0.51.1

---------

Co-authored-by: Copilot <[email protected]>
Co-authored-by: ericnordelo <[email protected]>

* Update contract sizes benchmark (#1559)

Co-authored-by: ericnordelo <[email protected]>

* Remove the coverage_incompatible feature (#1558)

* feat: update to 0.51.0

* feat: update CHANGELOG

* Update packages/testing/src/common.cairo

Co-authored-by: Copilot <[email protected]>

* feat: remove the feature

* feat: update to 0.51.1

---------

Co-authored-by: Copilot <[email protected]>

* chore: remove repetitive word in comment (#1561)

Signed-off-by: vastonus <[email protected]>

* Chore(deps): Bump crate-ci/typos from 1.38.1 to 1.39.0 (#1562)

Bumps [crate-ci/typos](https://github.com/crate-ci/typos) from 1.38.1 to 1.39.0.
- [Release notes](https://github.com/crate-ci/typos/releases)
- [Changelog](https://github.com/crate-ci/typos/blob/master/CHANGELOG.md)
- [Commits](crate-ci/typos@80c8a49...07d900b)

---
updated-dependencies:
- dependency-name: crate-ci/typos
  dependency-version: 1.39.0
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump snforge to 0.51.2 (#1563)

* Bump snforge version to 0.51.2

* Add changelog entry for snforge update to 0.51.2

* Bump Cairo to v2.13.1 (#1565)

* Update contract sizes benchmark (#1566)

Co-authored-by: immrsd <[email protected]>

* feat: improve logic

* feat: apply review comments

* feat: update CHANGELOG

---------

Signed-off-by: dependabot[bot] <[email protected]>
Signed-off-by: vastonus <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Copilot <[email protected]>
Co-authored-by: ericnordelo <[email protected]>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: vastonus <[email protected]>
Co-authored-by: immrsd <[email protected]>
immrsd added a commit that referenced this pull request Dec 25, 2025
@immrsd @ericnordelo
@dependabot @github-actions @vastonus
Merge v3.0.0 into main (#1622)
8de5342 
* Improve `AccessControlDefaultAdminRules` admin transfer delay change logic (#1567)

* Chore(deps): Bump stefanzweifel/git-auto-commit-action (#1553)

Bumps [stefanzweifel/git-auto-commit-action](https://github.com/stefanzweifel/git-auto-commit-action) from 6.0.1 to 7.0.0.
- [Release notes](https://github.com/stefanzweifel/git-auto-commit-action/releases)
- [Changelog](https://github.com/stefanzweifel/git-auto-commit-action/blob/master/CHANGELOG.md)
- [Commits](stefanzweifel/git-auto-commit-action@778341a...28e16e8)

---
updated-dependencies:
- dependency-name: stefanzweifel/git-auto-commit-action
  dependency-version: 7.0.0
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Update foundry to 0.51.0 (#1555)

* feat: update to 0.51.0

* feat: update CHANGELOG

* Update packages/testing/src/common.cairo

Co-authored-by: Copilot <[email protected]>

* feat: update to 0.51.1

---------

Co-authored-by: Copilot <[email protected]>

* Release openzeppelin_testing v6.0.0 (#1556)

* feat: update to 0.51.0

* feat: update CHANGELOG

* Update packages/testing/src/common.cairo

Co-authored-by: Copilot <[email protected]>

* feat: update CHANGELOG

* Bump openzeppelin_testing version to 6.0.0 and update docs

* feat: update to 0.51.1

---------

Co-authored-by: Copilot <[email protected]>
Co-authored-by: ericnordelo <[email protected]>

* Update contract sizes benchmark (#1559)

Co-authored-by: ericnordelo <[email protected]>

* Remove the coverage_incompatible feature (#1558)

* feat: update to 0.51.0

* feat: update CHANGELOG

* Update packages/testing/src/common.cairo

Co-authored-by: Copilot <[email protected]>

* feat: remove the feature

* feat: update to 0.51.1

---------

Co-authored-by: Copilot <[email protected]>

* chore: remove repetitive word in comment (#1561)

Signed-off-by: vastonus <[email protected]>

* Chore(deps): Bump crate-ci/typos from 1.38.1 to 1.39.0 (#1562)

Bumps [crate-ci/typos](https://github.com/crate-ci/typos) from 1.38.1 to 1.39.0.
- [Release notes](https://github.com/crate-ci/typos/releases)
- [Changelog](https://github.com/crate-ci/typos/blob/master/CHANGELOG.md)
- [Commits](crate-ci/typos@80c8a49...07d900b)

---
updated-dependencies:
- dependency-name: crate-ci/typos
  dependency-version: 1.39.0
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump snforge to 0.51.2 (#1563)

* Bump snforge version to 0.51.2

* Add changelog entry for snforge update to 0.51.2

* Bump Cairo to v2.13.1 (#1565)

* Update contract sizes benchmark (#1566)

Co-authored-by: immrsd <[email protected]>

* feat: improve logic

* feat: apply review comments

* feat: update CHANGELOG

---------

Signed-off-by: dependabot[bot] <[email protected]>
Signed-off-by: vastonus <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Copilot <[email protected]>
Co-authored-by: ericnordelo <[email protected]>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: vastonus <[email protected]>
Co-authored-by: immrsd <[email protected]>

* Add a note clarifying EIP-6372 clock bounds (#1600)

* Remove unused NEGATIVE_FEE error code (#1599)

* Clarify `ERC-4626` fees flow (#1602)

* Add a note to ERC4626HooksTrait

* Add clarifying comments to ERC4626 mocks

* Clarify `Governor` voting start (#1598)

* Add doc requirement clarifying that voting starts only after snapshot

* Mint initial supply at deployment for Votes mocks

* Add additional test cases for block-number-based governor voting

* Add additional test cases for timestamp-based governor voting

* Format files

* Governor: fix state at snapshot (#1606)

* Make state resolving logic consistent with Solidity

* Fix timestamp in Governor tests

* Add tests checking Governor Pending state at snapshot timestamp

* Add changelog entry

* Format files

* Fix changelog entry

* Extract duplicated logic into helper func in tests

* Reduce number of fuzzer runs

* Fix setup-snfoundry version in tests workflow

* Set number of fuzzer runs to 200

* Fix setup-snfoundry version

* Add v3.0.0 audit report (#1612)

* Add v3.0.0 updates to changelog

* Bump version to 3.0.0 and update presets page

* Update version of utils and interfaces modules to 2.1.0

* Update readmes

* Fix root dir in update-readme script

* Fix version in MetaTransactionV0 preset

---------

Signed-off-by: dependabot[bot] <[email protected]>
Signed-off-by: vastonus <[email protected]>
Co-authored-by: Eric Nordelo <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Copilot <[email protected]>
Co-authored-by: ericnordelo <[email protected]>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: vastonus <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ericnordelo ericnordelo ericnordelo approved these changes

Assignees

@immrsd immrsd

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@immrsd @ericnordelo