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

Drag-param does not work with multi-arity functions #1728

Open
Sleepful opened this issue Nov 26, 2023 · 2 comments
Open

Drag-param does not work with multi-arity functions #1728

Sleepful opened this issue Nov 26, 2023 · 2 comments
Labels
bug Something isn't working editor Related to clojure-lsp on a text editor
Projects

Comments

@Sleepful
Copy link

Sorry for not providing further information here, I think it is straightforward to reproduce.

Basically, the code action is not available once a function is multi-arity.

That's it, do tell me if it is just my machine and I can provide more info, but I suspect this will be easy to repro.

@Sleepful Sleepful added API Related to API / CLI bug Something isn't working labels Nov 26, 2023
@Sleepful
Copy link
Author

I am happy to help implement this, would require some initial pointers on where to change the code because I have not looked at the code base once.

@ericdallo ericdallo added editor Related to clojure-lsp on a text editor and removed API Related to API / CLI labels Dec 3, 2023
@ericdallo ericdallo added this to Low priority in clojure-lsp via automation Dec 3, 2023
@ericdallo
Copy link
Member

@Sleepful, the code for this refactoring is here, I suggest starting with a test to understand why it's not working, this feature is a little bit complex so there are corner cases like that one that needs to have extra care to work

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working editor Related to clojure-lsp on a text editor
Projects
clojure-lsp
Low priority
Development

No branches or pull requests

2 participants