Skip to content

Commit

Permalink
contracts-bedrock: bump forge-std 1fd874f0efdb711cb6807c4f4a000ed2805…
Browse files Browse the repository at this point in the history
…dc809 (#9043)

* contracts-bedrock: bump forge-std 1fd874f0efdb711cb6807c4f4a000ed2805dc809

Bumps `forge-std` to the latest commit that includes the `vm.dumpState`
cheatcode.

- foundry-rs/forge-std#499
- foundry-rs/foundry#6827

* contracts-bedrock: fix build
  • Loading branch information
tynes authored Jan 17, 2024
1 parent 15dca91 commit 5f55e84
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion packages/contracts-bedrock/lib/forge-std
2 changes: 1 addition & 1 deletion packages/contracts-bedrock/scripts/Deploy.s.sol
Original file line number Diff line number Diff line change
Expand Up @@ -115,7 +115,7 @@ contract Deploy is Deployer {
/// @notice The create2 salt used for deployment of the contract implementations.
/// Using this helps to reduce config across networks as the implementation
/// addresses will be the same across networks when deployed with create2.
function _implSalt() internal returns (bytes32) {
function _implSalt() internal view returns (bytes32) {
return keccak256(bytes(vm.envOr("IMPL_SALT", string("ethers phoenix"))));
}

Expand Down
2 changes: 1 addition & 1 deletion packages/contracts-bedrock/scripts/Deployer.sol
Original file line number Diff line number Diff line change
Expand Up @@ -624,7 +624,7 @@ abstract contract Deployer is Script {

/// @notice The context of the deployment is used to namespace the artifacts.
/// An unknown context will use the chainid as the context name.
function _getDeploymentContext() private returns (string memory) {
function _getDeploymentContext() private view returns (string memory) {
string memory context = vm.envOr("DEPLOYMENT_CONTEXT", string(""));
if (bytes(context).length > 0) {
return context;
Expand Down

0 comments on commit 5f55e84

Please sign in to comment.