From da02446859bbee179e4479f6209f40533a26f4e2 Mon Sep 17 00:00:00 2001 From: Ankit Jain Date: Fri, 15 May 2026 19:45:05 -0400 Subject: [PATCH] docs(cli): fix staging/daily channel example in ScaffoldingService comment The comment example listed `daily` as the identity that could pin a channel without a matching PSM rule. That is backwards: `daily` is registered unconditionally in `PackagingService`, while `staging` is the channel gated behind the staging feature flag. Update the example to `staging` so the comment matches the actual channel-registration behavior. Follow-up to PR #17120 review feedback. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --- src/Aspire.Cli/Scaffolding/ScaffoldingService.cs | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/Aspire.Cli/Scaffolding/ScaffoldingService.cs b/src/Aspire.Cli/Scaffolding/ScaffoldingService.cs index bf5f4a17b1f..79c4eea2c41 100644 --- a/src/Aspire.Cli/Scaffolding/ScaffoldingService.cs +++ b/src/Aspire.Cli/Scaffolding/ScaffoldingService.cs @@ -66,10 +66,10 @@ private async Task ScaffoldGuestLanguageAsync(ScaffoldContext context, Can // or NewCommand's identity-match against a registered Explicit channel — see // `CliTemplateFactory.EmptyTemplate.cs` for how `ScaffoldContext.Channel` is sourced). // Do NOT fall back to `CliExecutionContext.IdentityChannel`: an identity that isn't a - // registered channel (e.g. `daily` on a CLI without the staging feature flag, or `pr-` - // on a machine without the matching hive) would otherwise pin a channel name that no - // PSM rule can satisfy. When unset, `PrebuiltAppHostServer` aggregates sources from - // every registered channel so `aspire add` / `aspire restore` still find the right + // registered channel (e.g. `staging` on a CLI without the staging feature flag, or + // `pr-` on a machine without the matching hive) would otherwise pin a channel name + // that no PSM rule can satisfy. When unset, `PrebuiltAppHostServer` aggregates sources + // from every registered channel so `aspire add` / `aspire restore` still find the right // packages without a per-project pin. if (!string.IsNullOrEmpty(context.Channel)) {