chore: address comments

l1-contracts/src/core/Rollup.sol

@@ -39,6 +39,7 @@ contract Rollup is Leonidas, IRollup, ITestRollup {
 
   // @note  The number of slots within which a block must be proven
   //        This number is currently pulled out of thin air and should be replaced when we are not blind
+  // @todo  #8018
   uint256 public constant TIMELINESS_PROVING_IN_SLOTS = 100;
 
   IRegistry public immutable REGISTRY;

l1-contracts/src/core/sequencer_selection/Leonidas.sol

@@ -43,22 +43,24 @@ contract Leonidas is Ownable, ILeonidas {
     uint256 nextSeed;
   }
 
-  // @note @LHerskind - The multiple cause pain and suffering in the E2E tests as we introduce
+  // @note  @LHerskind  The multiple cause pain and suffering in the E2E tests as we introduce
   //                    a timeliness requirement into the publication that did not exists before,
   //                    and at the same time have a setup that will impact the time at every tx
   //                    because of auto-mine. By using just 1, we can make our test work
   //                    but anything using an actual working chain would eat dung as simulating
   //                    transactions is slower than an actual ethereum slot.
   //
   //                    The value should be a higher multiple for any actual chain
+  // @todo  #8019
   uint256 public constant SLOT_DURATION = Constants.ETHEREUM_SLOT_DURATION * 1;
 
   // The duration of an epoch in slots
-  // @todo @LHerskind - This value should be updated when we are not blind.
+  // @todo  @LHerskind - This value should be updated when we are not blind.
+  // @todo  #8020
   uint256 public constant EPOCH_DURATION = 32;
 
   // The target number of validators in a committee
-  // @todo @LHerskind - This value should be updated when we are not blind.
+  // @todo #8021
   uint256 public constant TARGET_COMMITTEE_SIZE = EPOCH_DURATION;
 
   // The time that the contract was deployed