nixvim: simplify condition

Checking `config.programs ? nixvim` **and** `options.programs ? nixvim` is redundant. Also, checking in the `mkIf` will not avoid missing option errors, only `optionalAttrs` is useful for that.
danth · Jun 5, 2024 · 7098394 · 7098394
1 parent b7bde26
commit 7098394
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/modules/nixvim/nixvim.nix b/modules/nixvim/nixvim.nix
@@ -38,8 +38,8 @@
   };
 
   config = {
-    programs = lib.mkIf ((config.programs ? nixvim) && config.stylix.targets.nixvim.enable) (
-      lib.optionalAttrs (builtins.hasAttr "nixvim" options.programs) {
+    programs = lib.mkIf config.stylix.targets.nixvim.enable (
+      lib.optionalAttrs (config.programs ? nixvim) {
         nixvim = config.stylix.targets.nixvim.config;
       }
     );