From 304de78463e5cc7d0d328825a6439f5f6c9b0053 Mon Sep 17 00:00:00 2001
From: Ryan Hall <hall.ryan.r@gmail.com>
Date: Wed, 11 Sep 2024 16:12:55 -0400
Subject: [PATCH] move natspec from RMNRemote to IRMNV2

---
 contracts/src/v0.8/ccip/interfaces/IRMNV2.sol |  5 +++++
 contracts/src/v0.8/ccip/rmn/RMNRemote.sol     | 11 +++--------
 2 files changed, 8 insertions(+), 8 deletions(-)

diff --git a/contracts/src/v0.8/ccip/interfaces/IRMNV2.sol b/contracts/src/v0.8/ccip/interfaces/IRMNV2.sol
index 95f67a314d..555abfbb15 100644
--- a/contracts/src/v0.8/ccip/interfaces/IRMNV2.sol
+++ b/contracts/src/v0.8/ccip/interfaces/IRMNV2.sol
@@ -11,6 +11,11 @@ interface IRMNV2 {
     bytes32 s;
   }
 
+  /// @notice Verifies signatures of RMN nodes, on dest lane updates as provided in the CommitReport
+  /// @param offrampAddress is not inferred by msg.sender, in case the call is made through ARMProxy
+  /// @param destLaneUpdates must be well formed, and is a representation of the CommitReport received from the oracles
+  /// @param signatures must be sorted in ascending order by signer address
+  /// @dev Will revert if verification fails
   function verify(
     address offRampAddress,
     Internal.MerkleRoot[] memory merkleRoots,
diff --git a/contracts/src/v0.8/ccip/rmn/RMNRemote.sol b/contracts/src/v0.8/ccip/rmn/RMNRemote.sol
index 5c6b0239fe..b0828dac01 100644
--- a/contracts/src/v0.8/ccip/rmn/RMNRemote.sol
+++ b/contracts/src/v0.8/ccip/rmn/RMNRemote.sol
@@ -89,11 +89,7 @@ contract RMNRemote is OwnerIsCreator, ITypeAndVersion, IRMNV2 {
   // │                         Verification                         │
   // ================================================================
 
-  /// @notice Verifies signatures of RMN nodes, on dest lane updates as provided in the CommitReport
-  /// @param offrampAddress is not inferred by msg.sender, in case the call is made through ARMProxy
-  /// @param destLaneUpdates must be well formed, and is a representation of the CommitReport received from the oracles
-  /// @param signatures must be sorted in ascending order by signer address
-  /// @dev Will revert if verification fails
+  /// @inheritdoc IRMNV2
   function verify(
     address offrampAddress,
     Internal.MerkleRoot[] memory destLaneUpdates,
@@ -236,7 +232,7 @@ contract RMNRemote is OwnerIsCreator, ITypeAndVersion, IRMNV2 {
     return s_cursedSubjectsSequence;
   }
 
-  /// @notice If there is an active global or legacy curse, this function returns true.
+  /// @inheritdoc IRMNV2
   function isCursed() external view returns (bool) {
     if (s_cursedSubjectsSequence.length == 0) {
       return false;
@@ -245,8 +241,7 @@ contract RMNRemote is OwnerIsCreator, ITypeAndVersion, IRMNV2 {
       s_cursedSubjectsIndexPlusOne[LEGACY_CURSE_SUBJECT] > 0 || s_cursedSubjectsIndexPlusOne[GLOBAL_CURSE_SUBJECT] > 0;
   }
 
-  /// @notice If there is an active global curse, or an active curse for `subject`, this function returns true.
-  /// @param subject To check whether a particular chain is cursed, set to bytes16(uint128(chainSelector)).
+  /// @inheritdoc IRMNV2
   function isCursed(bytes16 subject) external view returns (bool) {
     if (s_cursedSubjectsSequence.length == 0) {
       return false;