kibit icon indicating copy to clipboard operation
kibit copied to clipboard

Incorrect suggestion when using reducers

Open Kineolyan opened this issue 3 years ago • 1 comments

When using reducer methods instead of standard map or filter, kibit seems to get confused. With the following code, kibit suggests to replace into [] by vec:

(into [] (r/filter even? (r/map inc (range 10))))

However, this is not valid as vec is not a supported collection producer, unlike into.

If you need, I can provide the small lein project I cooked to test this.

Additional info

Clojure version: 1.10.1 Kibit version via lein-kibit: 0.1.8

Kineolyan avatar Sep 20 '21 21:09 Kineolyan

Beyond this being an error, is there a way to tell kibit to ignore a function or a form? Currently, this is the only pattern that causes me issues. But this suggestion makes the whole lein kibit returns an error code, which makes it impossible to use as a CI check.

Kineolyan avatar Sep 20 '21 21:09 Kineolyan