taoensso / carmine

Redis client + message queue for Clojure
https://www.taoensso.com/carmine
Eclipse Public License 1.0
1.15k stars 130 forks source link

Clojure 1.11 preparations - parse-double and parse-long #262

Closed ieugen closed 1 year ago

ieugen commented 2 years ago

Hi,

Upgraded to Clojure 1.11.0-rc1 and got these warnings:

; WARNING: parse-long already refers to: #'clojure.core/parse-long in namespace: taoensso.carmine, being replaced by: #'taoensso.carmine/parse-long
; WARNING: parse-double already refers to: #'clojure.core/parse-double in namespace: taoensso.carmine, being replaced by: #'taoensso.carmine/parse-double
olttwa commented 1 year ago

@ptaoussanis alongwith the warnings mentioned above, I'm also seeing the following:

WARNING: abs already refers to: #'clojure.core/abs in namespace: taoensso.encore, being replaced by: #'taoensso.encore/abs
WARNING: update-keys already refers to: #'clojure.core/update-keys in namespace: io.aviso.exception, being replaced by: #'io.aviso.exception/update-keys

It'd be nice to have a clj-kondo lint step in CI

ptaoussanis commented 1 year ago

Hi there! These warnings can safely be ignored, they're an implementation detail of Carmine - and will be addressed with the next release 👍

ptaoussanis commented 1 year ago

Closing, will be addressed in forthcoming release.