AstroNvim / docs

Site for AstroNvim
https://docs.astronvim.com
23 stars 59 forks source link

JDTLS example is not quite right #128

Open s1n7ax opened 11 months ago

s1n7ax commented 11 months ago

Checklist

Neovim version (nvim -v)

0.9.4

Operating system/version

NixOs 23

Terminal/GUI

Alacritty

Describe the bug

In the example given in https://docs.astronvim.com/recipes/advanced_lsp/#java-nvim-jdtls a new folder is created every time neovim is executed. And it concatenates the cwd so a new folder will be created by the name of the cwd even if it's not a java project.

local project_name = vim.fn.fnamemodify(vim.fn.getcwd(), ":p:h:t")
local workspace_dir = vim.fn.stdpath "data" .. "/site/java/workspace-root/" .. project_name

os.execute("mkdir " .. workspace_dir)

image

Additionally, os.execute does not work when the shell is nushell. Error of that actually visible for 200ms. I had to record and go frame by frame to read the error message. vim.fn.mkdir() probably a good alternative.

image

Steps to Reproduce

.

Expected behavior

.

Screenshots

No response

Additional Context

No response

mehalter commented 11 months ago

Thanks for reporting! If you have a better solution definitely make a PR :) We are currently on a doc freeze while we prepare for the v4 release. But if you open something up or give a better solution here I can incorporate it. Also, you could take a look at the AstroCommunity java pack and see if that's a better solution. We could upstream that config