From 795ed23b356dc044cfb497a6189d588604a6c335 Mon Sep 17 00:00:00 2001
From: Nathan Whitaker <17734409+nathanwhit@users.noreply.github.com>
Date: Wed, 24 Jul 2024 16:37:13 -0700
Subject: fix(future): Emit `deno install` warning less often, suggest `deno
 install` in error message (#24706)

Two small changes:

- In our BYONM errors, suggest running `deno install` instead of `npm
install` if `DENO_FUTURE` is set
- Only emit warning about `deno install` changes if you do `deno install
<foo>` with deno_future unset
---
 cli/npm/byonm.rs | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

(limited to 'cli/npm')

diff --git a/cli/npm/byonm.rs b/cli/npm/byonm.rs
index d10bb6b2a..86c9badac 100644
--- a/cli/npm/byonm.rs
+++ b/cli/npm/byonm.rs
@@ -286,9 +286,14 @@ impl CliNpmResolver for ByonmCliNpmResolver {
       concat!(
         "Could not find \"{}\" in a node_modules folder. ",
         "Deno expects the node_modules/ directory to be up to date. ",
-        "Did you forget to run `npm install`?"
+        "Did you forget to run `{}`?"
       ),
       alias,
+      if *crate::args::DENO_FUTURE {
+        "deno install"
+      } else {
+        "npm install"
+      }
     );
   }
 
-- 
cgit v1.2.3