We are using upgrades CLI, and when doing storage layout checks, we get errors for
- Solady
- @openzeppelin/contracts/proxy/beacon/UpgradeableBeacon.sol:UpgradeableBeacon
And in our case, both are safe to use.
Is there a way to skip validation for some contracts in the inheritance tree? Thanks
Can you share examples of which contracts in Solady show errors?
I've opened https://github.com/OpenZeppelin/openzeppelin-upgrades/issues/1035 for this request to allow skipping contracts.
node_modules/solady/src/accounts/ERC6551.sol:86: Variable `_cachedChainId` is immutable and will be initialized on the implementation
If by design, annotate with '@custom:oz-upgrades-unsafe-allow state-variable-immutable'
Otherwise, consider a constant variable or use a mutable variable instead
https://zpl.in/upgrades/error-005
node_modules/solady/src/utils/EIP712.sol:43: Contract `EIP712` has a constructor
Define an initializer instead
https://zpl.in/upgrades/error-001
node_modules/solady/src/utils/EIP712.sol:25: Variable `_cachedThis` is immutable and will be initialized on the implementation
If by design, annotate with '@custom:oz-upgrades-unsafe-allow state-variable-immutable'
Otherwise, consider a constant variable or use a mutable variable instead
https://zpl.in/upgrades/error-005
node_modules/solady/src/utils/EIP712.sol:26: Variable `_cachedChainId` is immutable and will be initialized on the implementation
If by design, annotate with '@custom:oz-upgrades-unsafe-allow state-variable-immutable'
Otherwise, consider a constant variable or use a mutable variable instead
https://zpl.in/upgrades/error-005
node_modules/solady/src/utils/EIP712.sol:27: Variable `_cachedNameHash` is immutable and will be initialized on the implementation
If by design, annotate with '@custom:oz-upgrades-unsafe-allow state-variable-immutable'
Otherwise, consider a constant variable or use a mutable variable instead
https://zpl.in/upgrades/error-005
node_modules/solady/src/utils/EIP712.sol:28: Variable `_cachedVersionHash` is immutable and will be initialized on the implementation
If by design, annotate with '@custom:oz-upgrades-unsafe-allow state-variable-immutable'
Otherwise, consider a constant variable or use a mutable variable instead
https://zpl.in/upgrades/error-005
node_modules/solady/src/utils/EIP712.sol:30: Variable `_cachedDomainSeparator` is immutable and will be initialized on the implementation
If by design, annotate with '@custom:oz-upgrades-unsafe-allow state-variable-immutable'
Otherwise, consider a constant variable or use a mutable variable instead
https://zpl.in/upgrades/error-005
node_modules/solady/src/utils/UUPSUpgradeable.sol:33: Variable `__self` is immutable and will be initialized on the implementation
If by design, annotate with '@custom:oz-upgrades-unsafe-allow state-variable-immutable'
Otherwise, consider a constant variable or use a mutable variable instead
https://zpl.in/upgrades/error-005