feat(noUnusedFunctionParameters): add lint for unused function parame…

…ters, instead of using noUnusedVariables (#2899)

Co-authored-by: printfn <[email protected]>

CHANGELOG.md

@@ -303,6 +303,7 @@ our [guidelines for writing a good changelog entry](https://github.com/biomejs/b
 - Add [nursery/noUnmatchableAnbSelector](https://biomejs.dev/linter/rules/no-unmatchable-anb-selector). [#2706](https://github.com/biomejs/biome/issues/2706) Contributed by @togami2864
 - Add [nursery/useGenericFontNames](https://biomejs.dev/linter/rules/use-generic-font-names). [#2573](https://github.com/biomejs/biome/pull/2573) Contributed by @togami2864
 - Add [nursery/noYodaExpression](https://biomejs.dev/linter/rules/no-yoda-expression/). Contributed by @michellocana
+- Add [nursery/noUnusedFunctionParameters](https://biomejs.dev/linter/rules/no-unused-function-parameters/) Contributed by @printfn
 
 #### Enhancements
 

crates/biome_diagnostics_categories/src/categories.rs

@@ -138,6 +138,7 @@ define_categories! {
     "lint/nursery/noUnknownSelectorPseudoElement": "https://biomejs.dev/linter/rules/no-unknown-selector-pseudo-element",
     "lint/nursery/noUnknownUnit": "https://biomejs.dev/linter/rules/no-unknown-unit",
     "lint/nursery/noUnmatchableAnbSelector": "https://biomejs.dev/linter/rules/no-unmatchable-anb-selector",
+    "lint/nursery/noUnusedFunctionParameters": "https://biomejs.dev/linter/rules/no-unused-function-parameters",
     "lint/nursery/noUselessStringConcat": "https://biomejs.dev/linter/rules/no-useless-string-concat",
     "lint/nursery/noUselessUndefinedInitialization": "https://biomejs.dev/linter/rules/no-useless-undefined-initialization",
     "lint/nursery/noYodaExpression": "https://biomejs.dev/linter/rules/no-yoda-expression",

crates/biome_js_analyze/src/lint/correctness/no_unused_variables.rs

@@ -30,6 +30,10 @@ declare_rule! {
     /// If you want to report unused imports,
     /// enable [noUnusedImports](https://biomejs.dev/linter/rules/no-unused-imports/).
     ///
+    /// From `v1.9.0`, the rule won't check unused function parameters any more.
+    /// Users should switch to
+    /// [noUnusedFunctionParameters](https://biomejs.dev/linter/rules/no-unused-function-parameters/)
+    ///
     /// ## Examples
     ///
     /// ### Invalid
@@ -65,7 +69,7 @@ declare_rule! {
     /// export function f<T>() {}
     /// ```
     ///
-    /// # Valid
+    /// ### Valid
     ///
     /// ```js
     /// function foo(b) {
@@ -99,7 +103,7 @@ declare_rule! {
     }
 }
 
-/// Suggestion if the bindnig is unused
+/// Suggestion if the binding is unused
 #[derive(Debug)]
 pub enum SuggestedFix {
     /// No suggestion will be given