chore: code suggestions

crates/rome_js_analyze/src/analyzers/nursery/use_blank_target.rs

@@ -30,17 +30,24 @@ declare_rule! {
     /// ```jsx,expect_diagnostic
     /// <a href='http://external.link' target='_blank'>child</a>
     /// ```
+    ///
     /// ```jsx,expect_diagnostic
     /// <a href='http://external.link' target='_blank' rel="noopener">child</a>
     /// ```
+    ///
+    /// ```jsx,expect_diagnostic
+    /// <a {...props} href='http://external.link' target='_blank' rel="noopener">child</a>
+    /// ```
+    ///
     /// ```jsx,expect_diagnostic
     /// // case-insensitive
     /// <a href='http://external.link' target='_BlaNk'>child</a>
     /// ```
     /// ### Valid
     ///
     /// ```jsx
-    /// <a href='http://external.link' rel='noreferrer' target='_blank'>child</a>
+    /// let a = <a href='http://external.link' rel='noreferrer' target='_blank'>child</a>;
+    /// let a = <a href='http://external.link' target='_blank' rel="noopener" {...props}>child</a>;
     /// ```
     pub(crate) UseBlankTarget {
         version: "10.0.0",

crates/rome_js_analyze/src/analyzers/nursery/use_key_with_click_events.rs

@@ -40,6 +40,10 @@ declare_rule! {
     /// // this rule doesn't apply to user created component
     /// <MyComponent onClick={() => {}} />
     ///```
+    ///
+    /// ```jsx,
+    /// <div onClick={() => {}} {...spread}></div>
+    /// ```
     pub(crate) UseKeyWithClickEvents {
         version: "10.0.0",
         name: "useKeyWithClickEvents",

crates/rome_js_analyze/src/analyzers/nursery/use_valid_anchor.rs

@@ -235,15 +235,10 @@ impl Rule for UseValidAnchor {
             (None, Some(on_click_attribute)) => Some(UseValidAnchorState::CantBeAnchor(
                 on_click_attribute.syntax().text_trimmed_range(),
             )),
-            (None, None) => {
-                if !node.has_spread_attribute()? {
-                    Some(UseValidAnchorState::MissingHrefAttribute(
-                        node.syntax().text_trimmed_range(),
-                    ))
-                } else {
-                    None
-                }
-            }
+            (None, None) if !node.has_spread_attribute()? => Some(
+                UseValidAnchorState::MissingHrefAttribute(node.syntax().text_trimmed_range()),
+            ),
+            _ => None,
         }
     }