They started appearing on the latest bump, it looks like my
configuration started including `nvim-web-devicons` (see [1]).
I'll probably remove this configuration on the next nixpkgs bump (it's a
good canary to check that I *never* include icons in the future).
[1]: https://github.com/NixOS/nixpkgs/pull/382668
I finally figured out why I was getting the wrong indentation, turns out
it was an issue in `lsp-format.nvim`. With that fixed/worked around, I
can now rely completely on `bash-language-server` for formatting.
I'll also rely on `shfmt` automatically detecting the type of file, as
(Neo)Vim cannot be made to reliably set `ft=bash` for Bash scripts and
`ft=sh` for POSIX shell.
Finally, I removed spaces after redirections, I've now come around to
liking the default (no spaces) better.
Since `none-ls` has removed their `shellcheck` built-in. This actually
makes the diagnostics more robust to POSIX/non-POSIX scripts (the LSP
server detects it at runtime, which is more robust than the `ftdetect`
scripts).
Nice bonus: the shellcheck code is shown in the diagnostics message
without any configuration!
I'm not sure if I can configure `avoid-nullary-conditions` -- though it
seems like this check is broken at the moment (I couldn't get it to
trigger during my tests).
`none-ls` deprecated a lot of unmaintained builtins, or ones that they
find has been replaced by a compete LSP server.
This removes those deprecated builtins, or uses a shim until I migrate
to the relevant LSP configuration (for `bash-language-server`).
The plug-in now configures itself automatically.
I could also set `vim.g.skip_ts_context_commentstring_module = true`,
but that should be removed soon enough anyway, and I would forget to
remove it...
