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

Cannot Complete Java Classes #1329

Closed
stevemolitor opened this issue Oct 19, 2022 · 3 comments · Fixed by #1331
Closed

Cannot Complete Java Classes #1329

stevemolitor opened this issue Oct 19, 2022 · 3 comments · Fixed by #1331
Labels
editor Related to clojure-lsp on a text editor enhancement New feature or request
Projects

Comments

@stevemolitor
Copy link

stevemolitor commented Oct 19, 2022

Describe the bug

I cannot complete Java classes in clojure-lsp in Emacs. Regular clojure (ie map, filter, etc) completes just fine.

To Reproduce

Type java.io.F and trigger completion. You will not see any candidates.

Expected behavior

If I type java.io.F and trigger completion, I should see candidates.

Screenshots

Trying to trigger completion of Java classes in Emacs using clojure-lsp:

With cider it works as expected:

User details (please complete the following information):

  • OS: MacOs
  • Editor: Emacs 29
  • Version: clojure-lsp 2022.10.05-16.48.26-nightly, clj-kondo 2022.10.05

Additional context

I have an empty deps.edn file:

{}

I assumed that Java standard library stuff would be picked up by clojure-lsp automatically, as it is with cider. But maybe I have a classpath issue, with clojure-lsp?

Great project btw - amazing stuff.

@stevemolitor stevemolitor added bug Something isn't working editor Related to clojure-lsp on a text editor labels Oct 19, 2022
@ericdallo ericdallo added enhancement New feature or request and removed bug Something isn't working labels Oct 19, 2022
@ericdallo
Copy link
Member

We never had this feature because until recently we didn't have java analysis from clj-kondo, I think we can add this kind of completion now, but need to double check

@ericdallo ericdallo added this to Low priority in clojure-lsp via automation Oct 19, 2022
@ericdallo
Copy link
Member

This is a possible fix: #1331

clojure-lsp automation moved this from Low priority to Next release Oct 30, 2022
@ericdallo
Copy link
Member

This should be fixed, you can try a nightly build or wait for next release

@ericdallo ericdallo moved this from Next release to Done in clojure-lsp Nov 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
editor Related to clojure-lsp on a text editor enhancement New feature or request
Projects
Development

Successfully merging a pull request may close this issue.

2 participants