Hide private variables from evaluate

casey · Aug 10, 2024 · 710f95a · 710f95a
1 parent e591af7
commit 710f95a
Show file tree

Hide file tree

Showing 2 changed files with 25 additions and 6 deletions.
diff --git a/src/justfile.rs b/src/justfile.rs
@@ -164,12 +164,14 @@ impl<'src> Justfile<'src> {
           let width = scope.names().fold(0, |max, name| name.len().max(max));
 
           for binding in scope.bindings() {
-            println!(
-              "{0:1$} := \"{2}\"",
-              binding.name.lexeme(),
-              width,
-              binding.value
-            );
+            if !self.settings.allow_private_variables || binding.is_public() {
+              println!(
+                "{0:1$} := \"{2}\"",
+                binding.name.lexeme(),
+                width,
+                binding.value
+              );
+            }
           }
         }
 

diff --git a/tests/evaluate.rs b/tests/evaluate.rs
@@ -80,6 +80,21 @@ test! {
 
 test! {
   name:    evaluate_private,
+  justfile: "
+    set allow-private-variables
+
+    [private]
+    foo := 'one'
+    bar := 'two'
+    _baz := 'three'
+  ",
+  args:   ("--evaluate"),
+  stdout: "bar  := \"two\"\n",
+  status: EXIT_SUCCESS,
+}
+
+test! {
+  name:    evaluate_private_not_enabled,
   justfile: "
     [private]
     foo := 'one'
@@ -94,6 +109,8 @@ test! {
 test! {
   name:    evaluate_single_private,
   justfile: "
+    set allow-private-variables
+
     [private]
     foo := 'one'
     bar := 'two'