Skip to content

Fix vscodium shell to work like nvim shell.#665

Open
sterlind wants to merge 3 commits into
nix-community:mainfrom
sterlind:dev/star/fix-codium-devshell
Open

Fix vscodium shell to work like nvim shell.#665
sterlind wants to merge 3 commits into
nix-community:mainfrom
sterlind:dev/star/fix-codium-devshell

Conversation

@sterlind

Copy link
Copy Markdown

Unlike the nvim shell, the vscodium shell didn't specify any nixd server settings, so the example flake.nix didn't work. Fixed this by:

  • Pulling vscodium's user settings.json into a separate file, and filling it with the same contents as the config elaborated in nvim-lsp.nix.

  • Moved dev shell generation in the nixd repo's flake.nix to a common mkDevShell function for both nvim and vscodium (since the settings.json is sepecific to the hardcoded /tmp/NixOS_Home-Manager path.)

Ideally we'd read the nixd server settings from the same file for both nvim and vscodium, but I don't know Lua so I didn't want to touch it.

Unlike the nvim shell, the vscodium shell didn't specify any nixd server
settings, so the example `flake.nix` didn't work. Fixed this by:

- Pulling vscodium's user settings.json into a separate file, and
  filling it with the same contents as the config elaborated in nvim-lsp.nix.

- Moved dev shell generation in the nixd repo's `flake.nix` to a common
  `mkDevShell` function for both nvim and vscodium (since the
  `settings.json` is sepecific to the hardcoded
  `/tmp/NixOS_Home-Manager` path.)

Ideally we'd read the nixd server settings from the same file for both
nvim and vscodium, but I don't know Lua so I didn't want to touch it.
Comment thread flake.nix
Comment thread nixd/docs/editors/vscode-settings.json Outdated
Comment thread .gitignore Outdated
@sterlind sterlind requested a review from inclyc April 11, 2025 23:07
@inclyc inclyc force-pushed the main branch 2 times, most recently from ed5029e to 6707401 Compare January 2, 2026 12:54
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