Skip to content

Update predeployed contracts to OpenZeppelin v0.20.0#695

Merged
FabijanC merged 6 commits intomainfrom
oz-0.20.0
Feb 12, 2025
Merged

Update predeployed contracts to OpenZeppelin v0.20.0#695
FabijanC merged 6 commits intomainfrom
oz-0.20.0

Conversation

@FabijanC
Copy link
Copy Markdown
Contributor

@FabijanC FabijanC commented Feb 12, 2025 •
edited
Loading

Usage related changes

Development related changes

  • Slower test execution due to significantly larger (x10) contract artifact for account and ERC20. Likely also impacts users.
  • Compiled locally using Scarb:
$ scarb --version
scarb 2.9.2 (5070ff374 2024-12-11)
cairo: 2.9.2 (https://crates.io/crates/cairo-lang-compiler/2.9.2)
sierra: 1.6.0

Checklist:

  • Checked out the contribution guidelines
  • Applied formatting - ./scripts/format.sh
  • No linter errors - ./scripts/clippy_check.sh
  • No unused dependencies - ./scripts/check_unused_deps.sh
  • No spelling errors - ./scripts/check_spelling.sh
  • Performed code self-review
  • Rebased to the latest commit of the target branch (or merged it into my branch)
    • Once you make the PR reviewable, please avoid force-pushing
  • Updated the docs if needed - ./website/README.md
  • Linked the issues resolvable by this PR - linking info
  • Updated the tests if needed; all passing - execution info

@FabijanC FabijanC marked this pull request as ready for review February 12, 2025 15:53
@FabijanC FabijanC merged commit d0c7322 into main Feb 12, 2025
@FabijanC FabijanC deleted the oz-0.20.0 branch February 12, 2025 16:04
FabijanC added a commit that referenced this pull request Feb 24, 2025
@FabijanC
Document the correct predeployed account; related: #695 [skip ci]
0506d94
@FabijanC FabijanC mentioned this pull request Mar 3, 2025
Closed
10 tasks
FabijanC added a commit that referenced this pull request Apr 8, 2025
@FabijanC
Replace OZ ERC20 0.20.0 with 0.8.1: partly revert d0c7322 (#695)
58c9573
@FabijanC FabijanC mentioned this pull request Apr 8, 2025
Merged
10 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Update OpenZeppelin contracts to a version newer than 0.8.1

1 participant

@FabijanC