git: use attrsOf instead of attrs

This makes sure that values added to

    programs.git.aliases

or

    programs.git.extraConfig

are merged as expected.

Also add a few option examples.

modules/programs/git.nix

@@ -6,6 +6,12 @@ let
 
   cfg = config.programs.git;
 
+  gitIniType = with types;
+    let
+      primitiveType = either bool (either int str);
+    in
+      attrsOf (attrsOf primitiveType);
+
   signModule = types.submodule {
     options = {
       key = mkOption {
@@ -80,8 +86,9 @@ in
       };
 
       aliases = mkOption {
-        type = types.attrs;
+        type = types.attrsOf types.str;
         default = {};
+        example = { co = "checkout"; };
         description = "Git aliases to define.";
       };
 
@@ -92,13 +99,16 @@ in
       };
 
       extraConfig = mkOption {
-        type = types.either types.attrs types.lines;
+        type = types.either types.lines gitIniType;
         default = {};
+        example = {
+          core = { whitespace = "trailing-space,space-before-tab"; };
+        };
         description = "Additional configuration to add.";
       };
 
       iniContent = mkOption {
-        type = types.attrsOf types.attrs;
+        type = gitIniType;
         internal = true;
       };
 

tests/modules/programs/git-expected.conf

@@ -1,11 +1,13 @@
 [alias]
 a1=foo
-a2=bar
+a2=baz
 
 [commit]
 gpgSign=true
 
 [extra]
+boolean=true
+integer=38
 name=value
 
 [gpg]

tests/modules/programs/git.nix

@@ -4,33 +4,41 @@ with lib;
 
 {
   config = {
-    programs.git = {
+    programs.git = mkMerge [
-      enable = true;
+      {
-      aliases = {
+        enable = true;
-        a1 = "foo";
+        aliases = {
-        a2 = "bar";
+          a1 = "foo";
-      };
+          a2 = "bar";
-      extraConfig = {
-        extra = {
-          name = "value";
         };
-      };
+        extraConfig = {
-      ignores = [ "*~" "*.swp" ];
+          extra = {
-      includes = [
+            name = "value";
-        { path = "~/path/to/config.inc"; }
+          };
-        {
+        };
-          path = "~/path/to/conditional.inc";
+        ignores = [ "*~" "*.swp" ];
-          condition = "gitdir:~/src/dir";
+        includes = [
-        }
+          { path = "~/path/to/config.inc"; }
-      ];
+          {
-      signing = {
+            path = "~/path/to/conditional.inc";
-        gpgPath = "path-to-gpg";
+            condition = "gitdir:~/src/dir";
-        key = "00112233445566778899AABBCCDDEEFF";
+          }
-        signByDefault = true;
+        ];
-      };
+        signing = {
-      userEmail = "user@example.org";
+          gpgPath = "path-to-gpg";
-      userName = "John Doe";
+          key = "00112233445566778899AABBCCDDEEFF";
-    };
+          signByDefault = true;
+        };
+        userEmail = "user@example.org";
+        userName = "John Doe";
+      }
+
+      {
+        aliases.a2 = mkForce "baz";
+        extraConfig.extra.boolean = true;
+        extraConfig.extra.integer = 38;
+      }
+    ];
 
     nmt.script = ''
       assertFileExists home-files/.config/git/config