From 130a66bce7d5af1727f618f6f358a87d24b767d3 Mon Sep 17 00:00:00 2001
From: Matt Sturgeon <matt@sturgeon.me.uk>
Date: Mon, 17 Jun 2024 01:20:07 +0100
Subject: [PATCH] plugins/ltex-extra: minor options cleanup

- Remove stringified default values
- Use an enum for log levels
- Remove adding package (done by mkNeovimPlugin)
---
 plugins/languages/ltex-extra.nix | 23 ++++++++++-------------
 1 file changed, 10 insertions(+), 13 deletions(-)

diff --git a/plugins/languages/ltex-extra.nix b/plugins/languages/ltex-extra.nix
index 28c33db4..123cb74f 100644
--- a/plugins/languages/ltex-extra.nix
+++ b/plugins/languages/ltex-extra.nix
@@ -28,21 +28,20 @@ helpers.neovim-plugin.mkNeovimPlugin config {
       Whether to load dictionaries on startup.
     '';
 
-    load_langs = helpers.defaultNullOpts.mkNullable (types.listOf types.str) ''["en-US"]'' ''
+    load_langs = helpers.defaultNullOpts.mkListOf types.str [ "en-US" ] ''
       Languages for witch dicionnaries will be loaded.
       See `plugins.lsp.servers.ltex.languages` for possible values.
     '';
 
-    log_level = helpers.defaultNullOpts.mkStr "none" ''
-      Log level. Possible values:
-      - "none"
-      - "trace"
-      - "debug"
-      - "info"
-      - "warn"
-      - "error"
-      - "fatal"
-    '';
+    log_level = helpers.defaultNullOpts.mkEnumFirstDefault [
+      "none"
+      "trace"
+      "debug"
+      "info"
+      "warn"
+      "error"
+      "fatal"
+    ] "Log level.";
   };
 
   extraConfig = cfg: {
@@ -51,8 +50,6 @@ helpers.neovim-plugin.mkNeovimPlugin config {
       You should set `plugins.lsp.enable = true` to make use of the LTeX_extra plugin's features.
     '';
 
-    extraPlugins = [ cfg.package ];
-
     plugins.lsp = {
       servers.ltex = {
         # Enable the ltex language server