nixvim icon indicating copy to clipboard operation
nixvim copied to clipboard

none-ls: updating + refactoring

Open MattSturgeon opened this issue 1 year ago • 4 comments

Overview issue to track specific issues, PRs, and ideas related to updating and refactoring none-ls.

  • [x] Switch to mkNeovimPlugin (move options to settings)
    • [x] #1727
  • [ ] Move sources.*.withArgs to sources.*.settings
    • [ ] #1724
    • [ ] Use either strLua submodule to allow a rename-alias
    • [ ] Include "filetypes" from builtins.json in option types/defaults/descriptions?
  • [ ] Consider how enableLspFormat should work with rfc42 settings?
    • [ ] When enabled, it sets on_attach default to require('lsp-format').on_attach.
    • [ ] Maybe we should just suggest users include this in their on_attach manually?
    • [x] Current description is misleading: #1690
  • [ ] Deprecate auto-enabling gitsigns?
    • [ ] Show a warning if there are not multiple definitions of plugins.gitsigns.enable.
    • [ ] Eventually, show a warning instead of auto-enabling.
  • [ ] Chore: attempt re-enabling disabled sources tests
  • [ ] Support "custom" sources submodules
    • [ ] Custom sources need to be registered first
    • [ ] Submodule shape: { method, generator, filetypes[], name?, condition? }.
  • [x] General refactoring
    • [x] Use a mkModule-style helper: #1853
    • [x] Move "builtin sources packages" list into a dedicated file
    • [x] Simplify pkg states: can be derivation or bool, don't need null.
    • [x] Move builtinPackages out of sync with builtins.json errors out of module and into a CI check
  • [x] Replace IFD with update script https://github.com/nix-community/nixvim/pull/1814

General notes and discussion can be had on this issue, but I'd encourage opening discussions or issues for specific issues. These can be linked in the relevant checklist above.

Tagging @traxys because he's done most of the work on the none-ls module so far.

MattSturgeon avatar Jun 15 '24 23:06 MattSturgeon