Skip to content

Commit

Permalink
chore: Apply suggestions from code review
Browse files Browse the repository at this point in the history
Co-authored-by: Vladimir Gorkavenko <[email protected]>
  • Loading branch information
dgusakov and vgorkavenko authored Jan 10, 2025
1 parent 73f281b commit ffaad65
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 5 deletions.
6 changes: 3 additions & 3 deletions script/DeployBase.s.sol
Original file line number Diff line number Diff line change
Expand Up @@ -270,7 +270,7 @@ abstract contract DeployBase is Script {
csm.grantRole(csm.PAUSE_ROLE(), gateSeal);
oracle.grantRole(oracle.PAUSE_ROLE(), gateSeal);
accounting.grantRole(accounting.PAUSE_ROLE(), gateSeal);
verifier.grantRole(accounting.PAUSE_ROLE(), gateSeal);
verifier.grantRole(verifier.PAUSE_ROLE(), gateSeal);

accounting.grantRole(
accounting.SET_BOND_CURVE_ROLE(),
Expand Down Expand Up @@ -299,8 +299,8 @@ abstract contract DeployBase is Script {
csm.grantRole(csm.DEFAULT_ADMIN_ROLE(), config.aragonAgent);
csm.revokeRole(csm.DEFAULT_ADMIN_ROLE(), deployer);

verifier.grantRole(csm.DEFAULT_ADMIN_ROLE(), config.aragonAgent);
verifier.revokeRole(csm.DEFAULT_ADMIN_ROLE(), deployer);
verifier.grantRole(verifier.DEFAULT_ADMIN_ROLE(), config.aragonAgent);
verifier.revokeRole(verifier.DEFAULT_ADMIN_ROLE(), deployer);

accounting.grantRole(
accounting.DEFAULT_ADMIN_ROLE(),
Expand Down
4 changes: 2 additions & 2 deletions script/DeployImplementationsBase.s.sol
Original file line number Diff line number Diff line change
Expand Up @@ -92,8 +92,8 @@ abstract contract DeployImplementationsBase is DeployBase {
gateSeal = _deployGateSeal(sealables);

verifier.grantRole(verifier.PAUSE_ROLE(), address(gateSeal));
verifier.grantRole(csm.DEFAULT_ADMIN_ROLE(), config.aragonAgent);
verifier.revokeRole(csm.DEFAULT_ADMIN_ROLE(), deployer);
verifier.grantRole(verifier.DEFAULT_ADMIN_ROLE(), config.aragonAgent);
verifier.revokeRole(verifier.DEFAULT_ADMIN_ROLE(), deployer);

JsonObj memory deployJson = Json.newObj();
deployJson.set("CSModuleImpl", address(csmImpl));
Expand Down

0 comments on commit ffaad65

Please sign in to comment.