Commit ec4c4f50 authored by Mark Tyneway's avatar Mark Tyneway

contracts-bedrock: make `implSalt()` internal

There is no reason for the `implSalt` method on the deploy
script to be public, so make it internal. This reduces the external
abi of the deploy script to prevent accidental leaky abstractions.
Co-authored-by: default avatarMatt Solomon <matt@mattsolomon.dev>
parent 79a8b1d7
...@@ -97,7 +97,7 @@ contract Deploy is Deployer { ...@@ -97,7 +97,7 @@ contract Deploy is Deployer {
/// @notice The create2 salt used for deployment of the contract implementations. /// @notice The create2 salt used for deployment of the contract implementations.
/// Using this helps to reduce config across networks as the implementation /// Using this helps to reduce config across networks as the implementation
/// addresses will be the same across networks when deployed with create2. /// addresses will be the same across networks when deployed with create2.
function implSalt() public returns (bytes32) { function implSalt() internal returns (bytes32) {
return keccak256(bytes(vm.envOr("IMPL_SALT", string("ethers phoenix")))); return keccak256(bytes(vm.envOr("IMPL_SALT", string("ethers phoenix"))));
} }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment