ctb: Apply suggestions from code review

packages/contracts-bedrock/scripts/Deploy.s.sol

@@ -401,7 +401,7 @@ contract Deploy is Deployer {
             _l2Oracle: l2OutputOracle,
             _guardian: guardian,
             _paused: true,
-            _config: systemConfig
+            _systemConfig: systemConfig
         });
 
         require(address(portal.L2_ORACLE()) == address(l2OutputOracle));

packages/contracts-bedrock/src/L1/L1CrossDomainMessenger.sol

@@ -19,8 +19,8 @@ contract L1CrossDomainMessenger is CrossDomainMessenger, ISemver {
     OptimismPortal public immutable PORTAL;
 
     /// @notice Semantic version.
-    /// @custom:semver 1.5.0
-    string public constant version = "1.5.0";
+    /// @custom:semver 1.8.0
+    string public constant version = "1.8.0";
 
     /// @notice Constructs the L1CrossDomainMessenger contract.
     /// @param _portal Address of the OptimismPortal contract on this network.

packages/contracts-bedrock/src/L1/L1ERC721Bridge.sol

@@ -19,8 +19,8 @@ contract L1ERC721Bridge is ERC721Bridge, ISemver {
     mapping(address => mapping(address => mapping(uint256 => bool))) public deposits;
 
     /// @notice Semantic version.
-    /// @custom:semver 1.5.1
-    string public constant version = "1.5.1";
+    /// @custom:semver 1.5.0
+    string public constant version = "1.5.0";
 
     /// @notice Constructs the L1ERC721Bridge contract.
     /// @param _messenger   Address of the CrossDomainMessenger on this network.

packages/contracts-bedrock/src/L1/L2OutputOracle.sol

@@ -15,13 +15,16 @@ contract L2OutputOracle is Initializable, ISemver {
     /// @notice The interval in L2 blocks at which checkpoints must be submitted.
     ///         Although this is immutable, it can safely be modified by upgrading the
     ///         implementation contract.
+    ///         Public getter is legacy and will be removed in the future. Use `submissionInterval`
+    ///         instead.
+    /// @custom:legacy
     uint256 public immutable SUBMISSION_INTERVAL;
 
     /// @notice The time between L2 blocks in seconds. Once set, this value MUST NOT be modified.
     uint256 public immutable L2_BLOCK_TIME;
 
     /// @notice The address of the challenger. Can be updated via upgrade. This will be removed in the
-    ///         future, use `proposer` instead.
+    ///         future, use `challenger` instead.
     /// @custom:legacy
     address public immutable CHALLENGER;
 

packages/contracts-bedrock/src/L1/OptimismPortal.sol

@@ -109,11 +109,11 @@ contract OptimismPortal is Initializable, ResourceMetering, ISemver {
     /// @param _l2Oracle Address of the L2OutputOracle contract.
     /// @param _guardian Address that can pause withdrawals.
     /// @param _paused Sets the contract's pausability state.
-    /// @param _config Address of the SystemConfig contract.
-    constructor(L2OutputOracle _l2Oracle, address _guardian, bool _paused, SystemConfig _config) {
+    /// @param _systemConfig Address of the SystemConfig contract.
+    constructor(L2OutputOracle _l2Oracle, address _guardian, bool _paused, SystemConfig _systemConfig) {
         L2_ORACLE = _l2Oracle;
         GUARDIAN = _guardian;
-        SYSTEM_CONFIG = _config;
+        SYSTEM_CONFIG = _systemConfig;
         initialize(_paused);
     }