merging symbols branch - Githubissues

ndonfris / fish-lsp

LSP implementation for the fish shell langauge 🐠

https://fish-lsp.dev

MIT License

97 stars 6 forks source link

merging symbols branch #5

Closed ndonfris closed 1 year ago

ndonfris commented 1 year ago

Using WorkspaceSymbol interface instead of DocumentSymbol
Changed hover docs for built-ins (now includes links)
Breaking server.ts
- onFormatProvider && onFormatRangeProvider
- onFoldProvider
- onCompletionProvider now supports ranges
- onRenameProvider
- onDefinitionProvider
- onRefrenceProvider
- added Diagnostics (coupled with code actions)
- added CodeActions
  - add missing end
  - remove extra end
  - '__' private autoload functions
  - remove duplicate functions
- added ConfigManager (api still needs completing)
  - need to further finalize client options
- Reworks to test-suite (removed useless tests and made easier to reuse)
  - LspDocument
  - Completion
  - fish-symbol-node.test.ts (contains base cases checks && docs for developers