search

l3nz / cli-matic

Compact, hands-free [sub]command line parsing library for Clojure.
Eclipse Public License 2.0
361 stars 29 forks source link

Linter warnings in 0.3.8 #77

Closed l3nz closed 4 years ago

l3nz commented 4 years ago

version

0.3.8

platform

JVM

problem

After upgrading, clj-kondo displays anomalies:

repro

$ lein clj-kondo
src/cli_matic/core.cljc:157:19: warning: use the idiom (seq x) rather than (not (empty? x))
src/cli_matic/core.cljc:423:7: warning: use the idiom (seq x) rather than (not (empty? x))
src/cli_matic/core.cljc:455:5: warning: Missing else branch.
src/cli_matic/core.cljc:575:5: warning: Missing else branch.
src/cli_matic/core.cljc:575:9: warning: use the idiom (seq x) rather than (not (empty? x))
src/cli_matic/optionals.clj:80:3: warning: Missing else branch.
src/cli_matic/platform.clj:33:3: warning: Missing else branch.
src/cli_matic/platform.cljs:41:3: warning: Missing else branch.
linting took 446ms, errors: 0, warnings: 8

expected behavior

I'd rather have no linter warnings.

l3nz commented 4 years ago 
$ lein clj-kondo
linting took 470ms, errors: 0, warnings: 0