diff options
| author | Nathan Whitaker <17734409+nathanwhit@users.noreply.github.com> | 2024-09-26 10:13:33 -0700 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2024-09-26 17:13:33 +0000 |
| commit | 543c687c3474233d674ba9938c40c9a33fab61e8 (patch) | |
| tree | 3aff7af00892a16ce0a0240011f45116df4ab49f /cli/args | |
| parent | 13c53d9727e0e529d04fd8b7709cb84b723fb0d8 (diff) | |
feat(install): warn repeatedly about not-run lifecycle scripts on explicit installs (#25878)
Currently we only warn once. With this PR, we continue to warn about
not-run scripts on explicit `deno install` (or cache). For `run` (or
other subcommands) we only warn the once, as we do currently.
Diffstat (limited to 'cli/args')
| -rw-r--r-- | cli/args/flags.rs | 2 | ||||
| -rw-r--r-- | cli/args/mod.rs | 6 |
2 files changed, 8 insertions, 0 deletions
diff --git a/cli/args/flags.rs b/cli/args/flags.rs index 74ccb512f..4e151d7d9 100644 --- a/cli/args/flags.rs +++ b/cli/args/flags.rs @@ -548,6 +548,8 @@ pub struct LifecycleScriptsConfig { pub allowed: PackagesAllowedScripts, pub initial_cwd: PathBuf, pub root_dir: PathBuf, + /// Part of an explicit `deno install` + pub explicit_install: bool, } #[derive(Debug, Clone, Eq, PartialEq, Default)] diff --git a/cli/args/mod.rs b/cli/args/mod.rs index 80b9afb24..a11d3bc10 100644 --- a/cli/args/mod.rs +++ b/cli/args/mod.rs @@ -1686,6 +1686,12 @@ impl CliOptions { allowed: self.flags.allow_scripts.clone(), initial_cwd: self.initial_cwd.clone(), root_dir: self.workspace().root_dir_path(), + explicit_install: matches!( + self.sub_command(), + DenoSubcommand::Install(_) + | DenoSubcommand::Cache(_) + | DenoSubcommand::Add(_) + ), } } } |