Merge pull request #1117 from rohaquinlop/fix-bug-testLicense

feat(build): #1099 fix testlicense bug
fluidattacks · Jul 4, 2023 · 91ee9fe · 91ee9fe
2 parents 4340e83 + 4f33658
commit 91ee9fe
Show file tree

Hide file tree

Showing 3 changed files with 16 additions and 5 deletions.
diff --git a/docs/src/api/builtins/test.md b/docs/src/api/builtins/test.md
@@ -4,15 +4,19 @@ Test the license of a project using [reuse](https://reuse.software/).
 
 Types:
 
-- testLicense: Empty attribute set.
+- testLicense:
+    - enable (`bool`): Optional.
+        Defaults to `false`.
 
 Example:
 
 === "makes.nix"
 
     ```nix
     {
-      testLicense = {};
+      testLicense = {
+        enable = true;
+      };
     }
     ```
 

diff --git a/makes.nix b/makes.nix
@@ -252,7 +252,9 @@
       };
     };
   };
-  testLicense = {};
+  testLicense = {
+    enable = true;
+  };
   testPython = {
     example = {
       python = "3.11";

diff --git a/src/evaluator/modules/test-license/default.nix b/src/evaluator/modules/test-license/default.nix
@@ -4,11 +4,16 @@
   ...
 }: {
   options = {
-    testLicense = {};
+    testLicense = {
+      enable = lib.mkOption {
+        default = false;
+        type = lib.types.bool;
+      };
+    };
   };
   config = {
     outputs = {
-      "/testLicense" = testLicense;
+      "/testLicense" = lib.mkIf config.testLicense.enable testLicense;
     };
   };
 }