Skip to content

feat(groovyls): use groovy-language-server from $PATH#4392

Merged
justinmk merged 1 commit intoneovim:masterfrom
briandipalma:patch-2
Apr 15, 2026
Merged

feat(groovyls): use groovy-language-server from $PATH#4392
justinmk merged 1 commit intoneovim:masterfrom
briandipalma:patch-2

Conversation

@briandipalma
Copy link
Copy Markdown
Contributor

I'd say most people will be using mason to install this. Anyway I'm not sure what was there before would work as the path would require the jar to be on your/neovim's path, I think, which won't be the case for something you download manually/or build locally so you'd have to move it there in which case creating a little shell script like mason does is hardly much extra effort.

image

@justinmk
Copy link
Copy Markdown
Member

justinmk commented Apr 15, 2026

most people will be using mason to install this

no, "mason" is never a valid rationale. but "find a program in the global $PATH" is.

the current config expects groovy-language-server-all.jar to be in CWD which is flimsy, so i'm just going to merge this because it's not any worse.

@justinmk justinmk changed the title feat(groovyls): Use mason path feat(groovyls): use groovy-language-server from $PATH Apr 15, 2026
@justinmk
Copy link
Copy Markdown
Member

the stylua issue should be fixed though

@justinmk justinmk merged commit 2a7c826 into neovim:master Apr 15, 2026
5 checks passed
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 this pull request may close these issues.

2 participants