fix(pixi.lock): reverted changes

fix(install.rs): add missing semicolon for proper syntax

refactor(upgrade.rs): remove no_activation flag to simplify upgrade function signature and reduce complexity

refactor(upgrade_all.rs): remove no_activation flag to align with upgrade function signature

test(test_main_cli.py): remove tests for no_activation flag to reflect removal of the flag in upgrade commands

src/cli/global/install.rs

@@ -231,7 +231,7 @@ pub(super) async fn create_executable_scripts(
             }
         } else {
             activation_script.clone()
-        }
+        };
 
         shell
             .run_command(

src/cli/global/upgrade.rs

@@ -30,10 +30,6 @@ pub struct Args {
     /// The platform to install the package for.
     #[clap(long, default_value_t = Platform::current())]
     platform: Platform,
-
-    /// Do not insert conda_prefix, path modifications, and activation script into the installed executable script.
-    #[clap(long)]
-    no_activation: bool,
 }
 
 impl HasSpecs for Args {
@@ -45,22 +41,14 @@ impl HasSpecs for Args {
 pub async fn execute(args: Args) -> miette::Result<()> {
     let config = Config::load_global();
     let specs = args.specs()?;
-    upgrade_packages(
-        specs,
-        config,
-        args.channels,
-        args.platform,
-        args.no_activation,
-    )
-    .await
+    upgrade_packages(specs, config, args.channels, args.platform).await
 }
 
 pub(super) async fn upgrade_packages(
     specs: IndexMap<PackageName, MatchSpec>,
     config: Config,
     cli_channels: ChannelsConfig,
     platform: Platform,
-    no_activation: bool,
 ) -> miette::Result<()> {
     let channel_cli = cli_channels.resolve_from_config(&config);
 
@@ -212,7 +200,7 @@ pub(super) async fn upgrade_packages(
                 records,
                 authenticated_client.clone(),
                 platform,
-                no_activation,
+                false,
             )
             .await?;
             pb.finish_with_message(format!("{} {}", console::style("Updated").green(), message));

src/cli/global/upgrade_all.rs

@@ -21,10 +21,6 @@ pub struct Args {
     /// The platform to install the package for.
     #[clap(long, default_value_t = Platform::current())]
     platform: Platform,
-
-    /// Do not insert conda_prefix, path modifications, and activation script into the installed executable script.
-    #[clap(long)]
-    no_activation: bool,
 }
 
 pub async fn execute(args: Args) -> miette::Result<()> {
@@ -42,12 +38,5 @@ pub async fn execute(args: Args) -> miette::Result<()> {
         );
     }
 
-    upgrade_packages(
-        specs,
-        config,
-        args.channels,
-        args.platform,
-        args.no_activation,
-    )
-    .await
+    upgrade_packages(specs, config, args.channels, args.platform).await
 }

tests/integration/test_main_cli.py

@@ -195,21 +195,9 @@ def test_global_install(pixi: Path) -> None:
     # Upgrade
     verify_cli_command([pixi, "global", "upgrade", "rattler-build"], ExitCode.SUCCESS)
 
-    # Upgrade with --no-activation
-    verify_cli_command(
-        [pixi, "global", "upgrade", "rattler-build", "--no-activation"],
-        ExitCode.SUCCESS,
-    )
-
     # Upgrade all
     verify_cli_command([pixi, "global", "upgrade-all"], ExitCode.SUCCESS)
 
-    # Upgrade all with --no-activation
-    verify_cli_command(
-        [pixi, "global", "upgrade-all", "--no-activation"],
-        ExitCode.SUCCESS,
-    )
-
     # List
     verify_cli_command([pixi, "global", "list"], ExitCode.SUCCESS, stderr_contains="rattler-build")