Compare commits

..

No commits in common. "ae901bb8cb6152eacd1c873a36fe1359daaac11d" and "a6a7bd37f314d2bff8212a452ae22bd6698c44e3" have entirely different histories.

3 changed files with 17 additions and 43 deletions

View file

@ -290,10 +290,8 @@
- `eslint_d` now checks for configuration files to load. - `eslint_d` now checks for configuration files to load.
- Fix an error where `eslint_d` fails to load. - Fix an error where `eslint_d` fails to load.
- Add required files support for linters under - Add required files support for linters under `vim.diagnostics.nvim-lint.linters.*.required_files`.
`vim.diagnostics.nvim-lint.linters.*.required_files`. - Add global function `nvf_lint` under `vim.diagnostics.nvim-lint.lint_function`.
- Add global function `nvf_lint` under
`vim.diagnostics.nvim-lint.lint_function`.
[Sc3l3t0n](https://github.com/Sc3l3t0n): [Sc3l3t0n](https://github.com/Sc3l3t0n):
@ -336,9 +334,3 @@
[Noah765](https://github.com/Noah765): [Noah765](https://github.com/Noah765):
- Add missing `flutter-tools.nvim` dependency `plenary.nvim`. - Add missing `flutter-tools.nvim` dependency `plenary.nvim`.
- Add necessary dependency of `flutter-tools.nvim` on lsp.
- Add the `vim.languages.dart.flutter-tools.flutterPackage` option.
[howird](https://github.com/howird):
- Change python dap adapter name from `python` to commonly expected `debugpy`.

View file

@ -13,7 +13,7 @@
inherit (lib.strings) optionalString; inherit (lib.strings) optionalString;
inherit (lib.nvim.lua) expToLua; inherit (lib.nvim.lua) expToLua;
inherit (lib.nvim.types) mkGrammarOption; inherit (lib.nvim.types) mkGrammarOption;
inherit (lib.nvim.dag) entryAfter; inherit (lib.nvim.dag) entryAnywhere;
cfg = config.vim.languages.dart; cfg = config.vim.languages.dart;
ftcfg = cfg.flutter-tools; ftcfg = cfg.flutter-tools;
@ -81,25 +81,16 @@ in {
description = "Enable flutter-tools for flutter support"; description = "Enable flutter-tools for flutter support";
}; };
flutterPackage = mkOption {
type = nullOr package;
default = pkgs.flutter;
description = "Flutter package, or null to detect the flutter path at runtime instead.";
};
enableNoResolvePatch = mkOption { enableNoResolvePatch = mkOption {
type = bool; type = bool;
default = false; default = true;
description = '' description = ''
Whether to patch flutter-tools so that it doesn't resolve Whether to patch flutter-tools so that it doesn't resolve
symlinks when detecting flutter path. symlinks when detecting flutter path.
::: {.note} This is required if you want to use a flutter package built with nix.
This is required if `flutterPackage` is set to null and the flutter If you are using a flutter SDK installed from a different source
package in your `PATH` was built with Nix. If you are using a flutter and encounter the error "`dart` missing from PATH", disable this option.
SDK installed from a different source and encounter the error "`dart`
missing from `PATH`", leave this option disabled.
:::
''; '';
}; };
@ -131,21 +122,19 @@ in {
}; };
}; };
config.vim = mkIf cfg.enable (mkMerge [ config = mkIf cfg.enable (mkMerge [
(mkIf cfg.treesitter.enable { (mkIf cfg.treesitter.enable {
treesitter.enable = true; vim.treesitter.enable = true;
treesitter.grammars = [cfg.treesitter.package]; vim.treesitter.grammars = [cfg.treesitter.package];
}) })
(mkIf cfg.lsp.enable { (mkIf cfg.lsp.enable {
lsp.lspconfig.enable = true; vim.lsp.lspconfig.enable = true;
lsp.lspconfig.sources.dart-lsp = servers.${cfg.lsp.server}.lspConfig; vim.lsp.lspconfig.sources.dart-lsp = servers.${cfg.lsp.server}.lspConfig;
}) })
(mkIf ftcfg.enable { (mkIf ftcfg.enable {
lsp.enable = true; vim.startPlugins = [
startPlugins = [
( (
if ftcfg.enableNoResolvePatch if ftcfg.enableNoResolvePatch
then "flutter-tools-patched" then "flutter-tools-patched"
@ -154,9 +143,8 @@ in {
"plenary-nvim" "plenary-nvim"
]; ];
pluginRC.flutter-tools = entryAfter ["lsp-setup"] '' vim.pluginRC.flutter-tools = entryAnywhere ''
require('flutter-tools').setup { require('flutter-tools').setup {
${optionalString (ftcfg.flutterPackage != null) "flutter_path = \"${ftcfg.flutterPackage}/bin/flutter\","}
lsp = { lsp = {
color = { -- show the derived colours for dart variables color = { -- show the derived colours for dart variables
enabled = ${boolToString ftcfg.color.enable}, -- whether or not to highlight color variables at all, only supported on flutter >= 2.10 enabled = ${boolToString ftcfg.color.enable}, -- whether or not to highlight color variables at all, only supported on flutter >= 2.10
@ -168,6 +156,7 @@ in {
capabilities = capabilities, capabilities = capabilities,
on_attach = default_on_attach; on_attach = default_on_attach;
flags = lsp_flags,
}, },
${optionalString cfg.dap.enable '' ${optionalString cfg.dap.enable ''
debugger = { debugger = {

View file

@ -5,7 +5,7 @@
... ...
}: let }: let
inherit (builtins) attrNames; inherit (builtins) attrNames;
inherit (lib.options) mkEnableOption mkOption literalMD; inherit (lib.options) mkEnableOption mkOption;
inherit (lib.modules) mkIf mkMerge; inherit (lib.modules) mkIf mkMerge;
inherit (lib.meta) getExe; inherit (lib.meta) getExe;
inherit (lib.lists) isList; inherit (lib.lists) isList;
@ -84,14 +84,7 @@ in {
}; };
format = { format = {
enable = enable = mkEnableOption "Go formatting" // {default = config.vim.languages.enableFormat;};
mkEnableOption "Go formatting"
// {
default = !cfg.lsp.enable && config.vim.languages.enableFormat;
defaultText = literalMD ''
disabled if Go LSP is enabled, otherwise follows {option}`vim.languages.enableFormat`
'';
};
type = mkOption { type = mkOption {
description = "Go formatter to use"; description = "Go formatter to use";