diff options
| author | Nathan Whitaker <17734409+nathanwhit@users.noreply.github.com> | 2024-07-10 11:03:41 -0700 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2024-07-10 18:03:41 +0000 |
| commit | 69afa8718f322cf2ef5f5cf5bcecb10f1122f490 (patch) | |
| tree | f6f6039283ff2338eea0420988077bc8ee4fe111 /cli/npm/managed/resolvers/local.rs | |
| parent | 26288cf2a97334e6d8e6697d0b858ad8582802b9 (diff) | |
chore: tweak warning message for un-run install scripts (#24508)
Previously when we printed out the packages that skipped install
scripts, we didn't prefix them with `npm:`. When you pass
`--allow-scripts` though, we require `npm:`, which means you can't just
copy paste the package name from the warning message.
Diffstat (limited to 'cli/npm/managed/resolvers/local.rs')
| -rw-r--r-- | cli/npm/managed/resolvers/local.rs | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/cli/npm/managed/resolvers/local.rs b/cli/npm/managed/resolvers/local.rs index 913cf986d..f73ba95ac 100644 --- a/cli/npm/managed/resolvers/local.rs +++ b/cli/npm/managed/resolvers/local.rs @@ -721,7 +721,7 @@ async fn sync_resolution_with_fs( }; let packages = packages_with_scripts_not_run .iter() - .map(|p| p.to_string()) + .map(|p| format!("npm:{p}")) .collect::<Vec<_>>() .join(", "); log::warn!("{}: Packages contained npm lifecycle scripts (preinstall/install/postinstall) that were not executed. |