-
-
Notifications
You must be signed in to change notification settings - Fork 227
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
LSP Unused Public Var -> Kondo Redundant Ignore #2722
Comments
Hi! Do you have the clj-kondo extension installed? If so, that could be the problem. |
No, I don't have clj-kondo installed. Can I do anything here to provide more info? My two team mates have the same issue. And we don't have any special Calva config or anything. I suspect many users are having the same problem. |
I suspect this is a clj-kondo issue. If so it is even more people affected. I could reproduce it in Calva with your description. Let's try repro it with clj-kondo alone:
Yeah, clj-kondo. And it is probably new, because otherwise I would have heard of it before. 😄 Check if it is reported on clj-kondo, otherwise, please report it. Let's keep this issue open until we have a resolution or some more info about it at least. |
I am assuming (defn ^:export foo [] 42) That tells clj-kondo (and humans) not to complain about it not being used in the project. |
Could you report to clj-kondo? If they start questioning me, I would just be a middleman relaying their questions to you. Thanks for the ^:export tip! My problem is solved. |
See this issue and this commit that's linked from that issue. |
It doesn't look like the change linked above has been released yet judging by the time of that commit and the time of the last kondo release, but it seems the solution will involve updating the kondo config. |
Hi! Thanks for Calva!!
With a tiny project like this:
deps.edn
src/main.clj
foo is marked with an info alert:
I click on the light bulb and it adds this line before foo:
Now that new line is marked with an info alert:
Versions:
What am I doing wrong?
Thanks!
The text was updated successfully, but these errors were encountered: