clostache icon indicating copy to clipboard operation
clostache copied to clipboard

Please replace :use with :refer

Open vemv opened this issue 4 years ago • 2 comments

Hi,

having Clostache in the dependecy chain causes the following warning to be printed:

WARNING: seqable? already refers to: #'clojure.core/seqable? in namespace: clostache.parser, being replaced by: #'clojure.core.incubator/seqable?

It can be fixed by simply replacing :use with :require - particularly when :use was already using its most conservative variant (:only).

As a further hint, there's https://github.com/bbatsov/clojure-style-guide/tree/e3be6b321447939371f2318e620a281ef4367a58#prefer-require-over-use

Thanks!

-V

vemv avatar Jul 05 '20 23:07 vemv

(Note that I can't submit a PR currently ⚖️)

vemv avatar Jul 05 '20 23:07 vemv