BetterThanTomorrow / calva

Clojure & ClojureScript Interactive Programming for VS Code
https://marketplace.visualstudio.com/items?itemName=betterthantomorrow.calva
Other
1.67k stars 217 forks source link

autocomplete for symbol in cljs file adds require from clj file #2308

Open zubairAmperity opened 1 year ago

zubairAmperity commented 1 year ago

The autocomplete doesn't include an symbol defined in a fn, but does include an example of an alias of a namespace not included in the project.

That namespace comes from a .clj file, and i'm working in a .cljs file, so it shouldn't even be an option for autocomplete

image

PEZ commented 1 year ago

Hi! That looks wrong. If there isn't an issue about this on clojure-lsp, I think @ericdallo would want one. If you can create a minimal reproduction, that would be extra awesome. 🙏

ericdallo commented 1 year ago

yes, clojure-lsp is aware of that during complete and that shouldn't happen, so we need a repro, feel free to create a issue when having one