Fork me on GitHub
Noah Bogart02:11:41

i would prefer to have them be automatic

Adam Helins17:11:50

(not (empty? ...))  issues a warning (use `seq` idiom instead) However, when this is the last form in a function for instance, the result is not the same (returning a boolean vs ?seq) I would suggest issuing this warning only when used in a conditional (`if` and friends) 🙂 (edited)


or suggest (boolean (seq ...)) ;)

☝️ 1

I don't know, clj-kondo just reflects what's in the docstring of empty?, empty? doesn't suggest the context of a condition

Adam Helins17:11:46

Yep or just disable that linter since it's not super crucial


as you see fit


you can also locally ignore it with #_:clj-kondo/ignore

Adam Helins17:11:50

Ah, is that new? I don't remember reading about that


it's been there for a while