Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Warnings about orchard dep shadowing clojure.core output when using Clojure 1.11.0 #19

Closed
stevejmp opened this issue Mar 30, 2022 · 2 comments · Fixed by #20
Closed

Comments

@stevejmp
Copy link
Contributor

stevejmp commented Mar 30, 2022

Expected Behavior

Use iced repl with no warnings in a project using Clojure 1.11.

Actual Behavior

WARNING: update-vals already refers to: #'clojure.core/update-vals in namespace: mrandersonicednrepl.orchard.v0v7v1.orchard.misc, being replaced by: #'mrandersonicednrepl.orchard.v0v7v1.orchard.misc/update-vals
WARNING: update-keys already refers to: #'clojure.core/update-keys in namespace: mrandersonicednrepl.orchard.v0v7v1.orchard.misc, being replaced by: #'mrandersonicednrepl.orchard.v0v7v1.orchard.misc/update-keys

There is also a similar warning about medley, but I reported that separately in #21 , because it's a different dependency and the fix hasn't yet been released.

Steps to Reproduce

  1. In a lein project, update the Clojure dep to 1.11.0
  2. Run iced repl
  3. Observe the warnings

More Info

Orchard already has a fix released. I'll submit a PR here to bump the version.

@stevejmp stevejmp changed the title Warning about orchard dep shadowing clojure.core output when using Clojure 1.11.0 Warnings about orchard dep shadowing clojure.core output when using Clojure 1.11.0 Mar 30, 2022
@liquidz
Copy link
Owner

liquidz commented Apr 1, 2022

@stevejmp Just released iced-nrepl v1.2.9 and vim-iced v3.8.4 which uses the latest iced-nrepl liquidz/vim-iced#394

Thank you so much!

@stevejmp
Copy link
Contributor Author

stevejmp commented Apr 4, 2022

こちらこそありがとうございます!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants