Skip to content

Conversation

@captainsafia
Copy link
Member

Avoid doing NuGet package prefetching when the do command is called.

Note: we're still doing the prefetch to check for CLI updates on every command (ref). We could probably avoid doing a NuGet call on every invocation to the CLI here.

Copilot AI review requested due to automatic review settings November 4, 2025 17:00
@github-actions
Copy link
Contributor

github-actions bot commented Nov 4, 2025

🚀 Dogfood this PR with:

⚠️ WARNING: Do not do this without first carefully reviewing the code of this PR to satisfy yourself it is safe.

curl -fsSL https://raw.githubusercontent.com/dotnet/aspire/main/eng/scripts/get-aspire-cli-pr.sh | bash -s -- 12669

Or

  • Run remotely in PowerShell:
iex "& { $(irm https://raw.githubusercontent.com/dotnet/aspire/main/eng/scripts/get-aspire-cli-pr.ps1) } 12669"

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds the "do" command to the list of runtime-only commands that should skip template package prefetching. The do command executes specific deployment pipeline steps and, like run, publish, and deploy, does not require template package metadata during its operation.

Key Changes

  • Added "do" to the list of runtime-only commands in IsRuntimeOnlyCommand() method in NuGetPackagePrefetcher.cs

@captainsafia captainsafia merged commit 6718a30 into main Nov 4, 2025
302 of 303 checks passed
@captainsafia captainsafia deleted the safia/fix-runtime-check branch November 4, 2025 19:49
@captainsafia
Copy link
Member Author

/backport to release/13.0

@github-actions
Copy link
Contributor

github-actions bot commented Nov 4, 2025

Started backporting to release/13.0: https://github.com/dotnet/aspire/actions/runs/19080994292

@dotnet-policy-service dotnet-policy-service bot added this to the 13.1 milestone Nov 4, 2025
@github-actions github-actions bot locked and limited conversation to collaborators Dec 6, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants