From 100f66733a75d7babf9d41595307bf11494f7ce0 Mon Sep 17 00:00:00 2001 From: vemv Date: Fri, 20 Aug 2021 22:48:40 +0200 Subject: [PATCH] Silence `:wrong-arity` false positives for honeysql for now See https://github.com/jonase/eastwood/issues/399 --- changes.md | 7 +++- resource/eastwood/config/third-party-libs.clj | 42 +++++++++++++++++++ 2 files changed, 48 insertions(+), 1 deletion(-) diff --git a/changes.md b/changes.md index 5c3d8c59..8e8e1f11 100644 --- a/changes.md +++ b/changes.md @@ -2,7 +2,7 @@ #### New -* Implement ability to omit a specific linter `:kind` +* Implement ability to omit a specific linter `:kind`. * Closes https://github.com/jonase/eastwood/issues/387 * [Documentation](https://github.com/jonase/eastwood#ignoring-linter-sub-kinds) * Now the `:second-arg-is-not-string` sub-kind for the `:suspicious-test` is disabled by default. @@ -10,6 +10,11 @@ * You can re-enable it by passing `:exclude-linters #{}` * [Learn more about sub-kind silencing](https://github.com/jonase/eastwood#ignoring-linter-sub-kinds) +#### Bugfixes + +* Silence `:wrong-arity` false positives for honeysql the time being. + * Part of https://github.com/jonase/eastwood/issues/399 + ## Changes from 0.9.4 to 0.9.6 #### Bugfixes diff --git a/resource/eastwood/config/third-party-libs.clj b/resource/eastwood/config/third-party-libs.clj index 210689a6..cdf7ada7 100644 --- a/resource/eastwood/config/third-party-libs.clj +++ b/resource/eastwood/config/third-party-libs.clj @@ -345,3 +345,45 @@ {:linter :constant-test :if-inside-macroexpansion-of #{'spec-tools.core/spec 'borkdude.dynaload/dynaload}}) + +(doseq [s '[honey.sql.helpers/add-index + honey.sql.helpers/alter-table + honey.sql.helpers/bulk-collect-into + honey.sql.helpers/clause-or-expression + honey.sql.helpers/create-extension + honey.sql.helpers/create-materialized-view + honey.sql.helpers/create-table + honey.sql.helpers/create-table-as + honey.sql.helpers/create-view + honey.sql.helpers/delete + honey.sql.helpers/delete-from + honey.sql.helpers/do-nothing + honey.sql.helpers/do-update-set + honey.sql.helpers/drop-column + honey.sql.helpers/fetch + honey.sql.helpers/filter + honey.sql.helpers/for + honey.sql.helpers/insert-into + honey.sql.helpers/into + honey.sql.helpers/limit + honey.sql.helpers/lock + honey.sql.helpers/offset + honey.sql.helpers/on-conflict + honey.sql.helpers/on-constraint + honey.sql.helpers/on-duplicate-key-update + honey.sql.helpers/refresh-materialized-view + honey.sql.helpers/rename-column + honey.sql.helpers/rename-table + honey.sql.helpers/select-distinct-on + honey.sql.helpers/set + honey.sql.helpers/truncate + honey.sql.helpers/update + honey.sql.helpers/values + honey.sql.helpers/with-columns + honey.sql.helpers/with-data + honey.sql.helpers/within-group]] + (disable-warning + {:linter :wrong-arity + :function-symbol s + :arglists-for-linting '([& _]) + :reason "Eastwood is currently unable to parse honeysql-style arglists, see https://github.com/jonase/eastwood/issues/399"}))