search

fhd / clostache

{{ mustache }} for Clojure
GNU Lesser General Public License v3.0
318 stars 62 forks source link

Please replace :use with :refer #63

Open vemv opened 4 years ago

vemv commented 4 years ago

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 commented 4 years ago

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

rajkrish commented 6 months ago

I am still seeing this problem in 2024. Please fix this open issue. Thank you.