Skip to content

Conversation

@ryanio
Copy link
Contributor

@ryanio ryanio commented Dec 11, 2025

Summary

  • Update openzeppelin-contracts submodule from v4.8.0 to v5.5.0 (latest stable)
  • Standardize all imports to use @openzeppelin/contracts/ format
  • Simplify foundry.toml to single remapping (only seaport-types needed, rest auto-detected)
  • Simplify CI workflow to just build, test, and format check

Changes

OpenZeppelin Update

Updated submodule to v5.5.0

Import Standardization

All OpenZeppelin imports now use the consistent @openzeppelin/contracts/ format

Simplified Remappings

remappings = [
    'seaport-types/=lib/seaport-types/',
]

All other remappings (forge-std, solady, @OpenZeppelin, solarray) are auto-detected by Forge.

Simplified CI Workflow

- Build
- Test  
- Format check (forge fmt --check)

Test plan

  • All 184 tests pass locally
  • CI pipeline passes

@ryanio ryanio force-pushed the fix/openzeppelin-imports branch 2 times, most recently from bdd5178 to 2521efb Compare December 11, 2025 16:01
- Update openzeppelin-contracts submodule from v4.8.0 to v5.5.0
- Standardize all imports to use @openzeppelin/contracts/ format
- Simplify foundry.toml to single remapping (seaport-types only, rest auto-detected)
- Simplify CI workflow to just build, test, and format check
@ryanio ryanio force-pushed the fix/openzeppelin-imports branch from 2521efb to 22aa5eb Compare December 11, 2025 16:03
@ryanio ryanio force-pushed the fix/openzeppelin-imports branch from 6c6d2a9 to 184d24d Compare December 11, 2025 16:07
@ryanio ryanio merged commit 704f9db into main Dec 11, 2025
6 checks passed
@ryanio ryanio deleted the fix/openzeppelin-imports branch December 22, 2025 18:56
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