From 5fd71628da7e62f11784e55953a08ba3930cf970 Mon Sep 17 00:00:00 2001
From: lucas-manuel <lucasmanuel.tech@gmail.com>
Date: Fri, 14 Jun 2024 09:25:01 -0400
Subject: [PATCH] chore: reduce diff

---
 src/Executor.sol | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/src/Executor.sol b/src/Executor.sol
index 6369379..e0524e1 100644
--- a/src/Executor.sol
+++ b/src/Executor.sol
@@ -52,9 +52,8 @@ contract Executor is IExecutor, AccessControl {
         _setRoleAdmin(SUBMISSION_ROLE, DEFAULT_ADMIN_ROLE);
         _setRoleAdmin(GUARDIAN_ROLE,   DEFAULT_ADMIN_ROLE);
 
-        // Necessary for self-referential calls to change configuration
-        _grantRole(DEFAULT_ADMIN_ROLE, address(this));
         _grantRole(DEFAULT_ADMIN_ROLE, msg.sender);
+        _grantRole(DEFAULT_ADMIN_ROLE, address(this));  // Necessary for self-referential calls to change configuration
     }
 
     /******************************************************************************************************************/
@@ -151,7 +150,7 @@ contract Executor is IExecutor, AccessControl {
     function cancel(uint256 actionsSetId) external override onlyRole(GUARDIAN_ROLE) {
         if (getCurrentState(actionsSetId) != ActionsSetState.Queued) revert OnlyQueuedActions();
 
-        ActionsSet storage actionsSet =_actionsSets[actionsSetId];
+        ActionsSet storage actionsSet = _actionsSets[actionsSetId];
         actionsSet.canceled = true;
 
         uint256 targetsLength = actionsSet.targets.length;