Update error message to refer to params & arity clause

tomdl89 · Nov 24, 2023 · 6dac281 · 6dac281
1 parent e44e734
commit 6dac281
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 7 deletions.
diff --git a/doc/linters.md b/doc/linters.md
@@ -857,7 +857,7 @@ be an unquoted symbol.
 
 *Example trigger:* `(fn :fn-name [x] (inc x))`.
 
-*Example message:* `First arg of fn should be a symbol, arg vector or body list`.
+*Example message:* `First arg of fn should be a symbol, params vector or arity clause`.
 
 *Config:*
 

diff --git a/src/clj_kondo/impl/analyzer.clj b/src/clj_kondo/impl/analyzer.clj
@@ -947,7 +947,7 @@
     filename
     expr
     :invalid-fn-name
-    "First arg of fn should be a symbol, arg vector or body list")))
+    "First arg of fn should be a symbol, params vector or arity clause")))
 
 (defn- def-fn? [{:keys [callstack]}]
   (let [[_ parent extra-parent] callstack]

diff --git a/test/clj_kondo/main_test.clj b/test/clj_kondo/main_test.clj
@@ -61,19 +61,19 @@
 
 (deftest invalid-fn-name-test
   (assert-submaps
-   '({:row 1, :col 1, :level :error, :message "First arg of fn should be a symbol, arg vector or body list"})
+   '({:row 1, :col 1, :level :error, :message "First arg of fn should be a symbol, params vector or arity clause"})
    (lint! "(fn \"fn-name\" [x] (inc x))"))
   (assert-submaps
-   '({:row 1, :col 6, :level :error, :message "First arg of fn should be a symbol, arg vector or body list"})
+   '({:row 1, :col 6, :level :error, :message "First arg of fn should be a symbol, params vector or arity clause"})
    (lint! "(map (fn 'symbol ([x] (inc x))) coll)"))
   (assert-submaps
-   '({:row 1, :col 7, :level :error, :message "First arg of fn should be a symbol, arg vector or body list"})
+   '({:row 1, :col 7, :level :error, :message "First arg of fn should be a symbol, params vector or arity clause"})
    (lint! "(-> 7 (fn [x] (inc x)))"))
   (assert-submaps
-   '({:row 1, :col 7, :level :error, :message "First arg of fn should be a symbol, arg vector or body list"})
+   '({:row 1, :col 7, :level :error, :message "First arg of fn should be a symbol, params vector or arity clause"})
    (lint! "(-> 7 #(inc %))"))
   (assert-submaps
-   '({:row 1, :col 1, :level :error, :message "First arg of fn should be a symbol, arg vector or body list"})
+   '({:row 1, :col 1, :level :error, :message "First arg of fn should be a symbol, params vector or arity clause"})
    (lint! "(fn* :fn-name [x] (inc x))"))
 
   (are [lint-form] (empty? lint-form)