From 7c63fb63b1517a95de950443d562cc71576ccc71 Mon Sep 17 00:00:00 2001
From: Casey Rodarmor <casey@rodarmor.com>
Date: Fri, 7 Feb 2025 10:47:41 -0800
Subject: [PATCH] Make `--list-submodules` require `--list` (#2622)

---
 src/config.rs | 2 +-
 tests/list.rs | 9 +++++++++
 2 files changed, 10 insertions(+), 1 deletion(-)

diff --git a/src/config.rs b/src/config.rs
index 9ed26fca00..3e17ed0c39 100644
--- a/src/config.rs
+++ b/src/config.rs
@@ -289,7 +289,7 @@ impl Config {
           .env("JUST_LIST_SUBMODULES")
           .help("List recipes in submodules")
           .action(ArgAction::SetTrue)
-          .env("JUST_LIST_SUBMODULES"),
+          .requires(cmd::LIST),
       )
       .arg(
         Arg::new(arg::NO_ALIASES)
diff --git a/tests/list.rs b/tests/list.rs
index 53a1c7372c..1d9377185f 100644
--- a/tests/list.rs
+++ b/tests/list.rs
@@ -474,3 +474,12 @@ fn unclosed_backticks() {
       ")
     .run();
 }
+
+#[test]
+fn list_submodules_requires_list() {
+  Test::new()
+    .arg("--list-submodules")
+    .stderr_regex("error: the following required arguments were not provided:\n  --list .*")
+    .status(2)
+    .run();
+}