Skip to content

Commit

Permalink
Fix empty version in deploy output file name. (#250)
Browse files Browse the repository at this point in the history
  • Loading branch information
kingster-will authored Sep 19, 2024
1 parent e51c932 commit ae0ab98
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 3 deletions.
3 changes: 2 additions & 1 deletion script/foundry/deployment/Main.s.sol
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,8 @@ contract Main is DeployHelper {
super.run(
seed, // create3 seed
false, // runStorageLayoutCheck
true // writeDeployments,
true, // writeDeployments,
VERSION
);
_writeDeployment(VERSION); // write deployment json to deployments/deployment-{chainId}.json
}
Expand Down
3 changes: 2 additions & 1 deletion script/foundry/utils/DeployHelper.sol
Original file line number Diff line number Diff line change
Expand Up @@ -157,9 +157,10 @@ contract DeployHelper is Script, BroadcastManager, JsonDeploymentHandler, Storag
/// @dev To use, run the following command (e.g. for Sepolia):
/// forge script script/foundry/deployment/Main.s.sol:Main --rpc-url $RPC_URL --broadcast --verify -vvvv

function run(uint256 create3SaltSeed_, bool runStorageLayoutCheck, bool writeDeploys_) public virtual {
function run(uint256 create3SaltSeed_, bool runStorageLayoutCheck, bool writeDeploys_, string memory version_) public virtual {
create3SaltSeed = create3SaltSeed_;
writeDeploys = writeDeploys_;
version = version_;

// This will run OZ storage layout check for all contracts. Requires --ffi flag.
if (runStorageLayoutCheck) super.run();
Expand Down
3 changes: 2 additions & 1 deletion test/foundry/utils/BaseTest.t.sol
Original file line number Diff line number Diff line change
Expand Up @@ -83,7 +83,8 @@ contract BaseTest is Test, DeployHelper, LicensingHelper {
super.run(
CREATE3_DEFAULT_SEED,
false, // runStorageLayoutCheck
false // writeDeploys
false, // writeDeploys
"test" // version
);

initLicensingHelper(address(pilTemplate), address(royaltyPolicyLAP), address(erc20));
Expand Down

0 comments on commit ae0ab98

Please sign in to comment.