Skip to content

Surface ASPIRE010 help link inline in warning text - #19375

Merged
Jose Perez Rodriguez (joperezr) merged 1 commit into
mainfrom
joperezr-aspire010-inline-help-link
Aug 15, 2026
Merged

Surface ASPIRE010 help link inline in warning text#19375
Jose Perez Rodriguez (joperezr) merged 1 commit into
mainfrom
joperezr-aspire010-inline-help-link

Conversation

@joperezr

Copy link
Copy Markdown
Member

Description

The MSBuild warning ASPIRE010 (emitted when an AppHost is configured with AspireUseCliBundle=false) carries its forward link only as a HelpLink metadata attribute pointing at https://aka.ms/aspire/diagnostics/aspire010.

MSBuild's HelpLink metadata is not rendered in terminal/console build output, so users who hit this warning in the console never see the link. This surfaces the URL inline in the warning Text so it appears in console output, while keeping the existing HelpLink metadata as-is.

Changes

  • src/Aspire.Hosting.AppHost/build/Aspire.Hosting.AppHost.in.targets: appended See https://aka.ms/aspire/diagnostics/aspire010 for more information. to the end of the ASPIRE010 warning Text. HelpLink, Code, Condition, and target name are unchanged. (Edited the .in.targets source template, not any generated .targets.)
  • docs/list-of-diagnostics.md: updated the ASPIRE010 row's description to match the new message text.

Existing MSBuild tests assert on substrings of the warning message and remain valid.

Note: the aka.ms link may not resolve to a real doc yet — the full aspire.dev doc will be wired up separately.

Contributes to #19368

MSBuild HelpLink metadata is not rendered in terminal/console build
output, so users who hit ASPIRE010 never saw the forward link. Append
the help URL inline to the warning Text so it appears in the console,
while keeping the existing HelpLink metadata. Keep the docs diagnostics
table in sync.

Contributes to #19368

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot AI balanced review requested due to automatic review settings August 14, 2026 02:52
@github-actions

Copy link
Copy Markdown
Contributor

🚀 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/microsoft/aspire/main/eng/scripts/get-aspire-cli-pr.sh | bash -s -- 19375

Or

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

@github-actions github-actions Bot added the needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners label Aug 14, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Tests selector (audit mode)

The full test matrix and all jobs still run in audit mode. The tests and jobs below are what selective CI would run under enforcement.

42 / 100 test projects · 2 jobs, from 1 changed file.

Selected test projects (42 / 100)

Aspire.EndToEnd.Tests, Aspire.Hosting.Analyzers.Tests, Aspire.Hosting.Azure.Kubernetes.Tests, Aspire.Hosting.Azure.Kusto.Tests, Aspire.Hosting.Azure.Tests, Aspire.Hosting.Blazor.Tests, Aspire.Hosting.Browsers.Tests, Aspire.Hosting.Containers.Tests, Aspire.Hosting.DevTunnels.Tests, Aspire.Hosting.Docker.Tests, Aspire.Hosting.Dotnet.Tests, Aspire.Hosting.DotnetTool.Tests, Aspire.Hosting.EntityFrameworkCore.Tests, Aspire.Hosting.Foundry.Tests, Aspire.Hosting.Garnet.Tests, Aspire.Hosting.GitHub.Models.Tests, Aspire.Hosting.Go.Tests, Aspire.Hosting.JavaScript.Tests, Aspire.Hosting.Kafka.Tests, Aspire.Hosting.Keycloak.Tests, Aspire.Hosting.Kubernetes.Tests, Aspire.Hosting.Maui.Tests, Aspire.Hosting.Milvus.Tests, Aspire.Hosting.MongoDB.Tests, Aspire.Hosting.MySql.Tests, Aspire.Hosting.Nats.Tests, Aspire.Hosting.OpenAI.Tests, Aspire.Hosting.Oracle.Tests, Aspire.Hosting.Orleans.Tests, Aspire.Hosting.PostgreSQL.Tests, Aspire.Hosting.Python.Tests, Aspire.Hosting.Qdrant.Tests, Aspire.Hosting.RabbitMQ.Tests, Aspire.Hosting.Radius.Tests, Aspire.Hosting.Redis.Tests, Aspire.Hosting.Seq.Tests, Aspire.Hosting.SqlServer.Tests, Aspire.Hosting.Testing.Tests, Aspire.Hosting.Tests, Aspire.Hosting.Valkey.Tests, Aspire.Hosting.Yarp.Tests, Aspire.Playground.Tests

Selected jobs (2)

extension-e2e, typescript-api-compat


How these were chosen — grouped by what changed

⚠️ 41 of the 42 selected test projects come from a single change — src/Aspire.Hosting.AppHost/build/Aspire.Hosting.AppHost.in.targets.

🔧 src/Aspire.Hosting.AppHost/build/Aspire.Hosting.AppHost.in.targets (changed source)
41 via the project graph

show 41

Aspire.Hosting.Analyzers.Tests, Aspire.Hosting.Azure.Kubernetes.Tests, Aspire.Hosting.Azure.Kusto.Tests, Aspire.Hosting.Azure.Tests, Aspire.Hosting.Blazor.Tests, Aspire.Hosting.Browsers.Tests, Aspire.Hosting.Containers.Tests, Aspire.Hosting.DevTunnels.Tests, Aspire.Hosting.Docker.Tests, Aspire.Hosting.Dotnet.Tests, Aspire.Hosting.DotnetTool.Tests, Aspire.Hosting.EntityFrameworkCore.Tests, Aspire.Hosting.Foundry.Tests, Aspire.Hosting.Garnet.Tests, Aspire.Hosting.GitHub.Models.Tests, Aspire.Hosting.Go.Tests, Aspire.Hosting.JavaScript.Tests, Aspire.Hosting.Kafka.Tests, Aspire.Hosting.Keycloak.Tests, Aspire.Hosting.Kubernetes.Tests, Aspire.Hosting.Maui.Tests, Aspire.Hosting.Milvus.Tests, Aspire.Hosting.MongoDB.Tests, Aspire.Hosting.MySql.Tests, Aspire.Hosting.Nats.Tests, Aspire.Hosting.OpenAI.Tests, Aspire.Hosting.Oracle.Tests, Aspire.Hosting.Orleans.Tests, Aspire.Hosting.PostgreSQL.Tests, Aspire.Hosting.Python.Tests, Aspire.Hosting.Qdrant.Tests, Aspire.Hosting.RabbitMQ.Tests, Aspire.Hosting.Radius.Tests, Aspire.Hosting.Redis.Tests, Aspire.Hosting.Seq.Tests, Aspire.Hosting.SqlServer.Tests, Aspire.Hosting.Testing.Tests, Aspire.Hosting.Tests, Aspire.Hosting.Valkey.Tests, Aspire.Hosting.Yarp.Tests, Aspire.Playground.Tests

📦 affected project Aspire.Hosting.AppHost
1 test: Aspire.EndToEnd.Tests

Job reasons

Job Triggered by
extension-e2e src/Aspire.Hosting.AppHost/build/Aspire.Hosting.AppHost.in.targets
• affected project Aspire.Hosting.TestUtilities
typescript-api-compat affected project Aspire.Hosting.TestUtilities

Selection computed for commit 89112ec.

@joperezr

Copy link
Copy Markdown
Member Author

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Surfaces the ASPIRE010 help URL directly in console warning output.

Changes:

  • Appends the diagnostic URL to the warning text.
  • Synchronizes the diagnostics documentation.
Show a summary per file
File Description
src/Aspire.Hosting.AppHost/build/Aspire.Hosting.AppHost.in.targets Adds the inline help URL.
docs/list-of-diagnostics.md Updates the documented warning text.

Review details

  • Files reviewed: 2/2 changed files
  • Comments generated: 1
  • Review effort level: Balanced

Comment thread src/Aspire.Hosting.AppHost/build/Aspire.Hosting.AppHost.in.targets
@github-actions

Copy link
Copy Markdown
Contributor

Retrying the failed CI jobs for this pull request from the CI run attempt. The rerun is being tracked in the rerun attempt.

Comment thread docs/list-of-diagnostics.md
@joperezr
Jose Perez Rodriguez (joperezr) merged commit 0cedb24 into main Aug 15, 2026
723 of 729 checks passed
@joperezr
Jose Perez Rodriguez (joperezr) deleted the joperezr-aspire010-inline-help-link branch August 15, 2026 17:54
@joperezr

Copy link
Copy Markdown
Member Author

/backport to release/13.5

@github-actions github-actions Bot added this to the 13.6 milestone Aug 15, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Started backporting to release/13.5 (link to workflow run)

@aspire-repo-bot

Copy link
Copy Markdown
Contributor

Pull request created: #1504

Generated by PR Documentation Check · auto · 47.6 AIC · ⌖ 15.4 AIC · ⊞ 19.6K

@aspire-repo-bot

Copy link
Copy Markdown
Contributor

📝 Documentation has been drafted in microsoft/aspire.dev#1504 targeting release/13.5.

Added a note to get-started/aspire-sdk.mdx describing when ASPIRE010 is emitted (opting out of AspireUseCliBundle) and that its warning text now includes the aka.ms/aspire/diagnostics/aspire010 help link inline so it's visible in console/terminal build output. This closes a gap where ASPIRE009/CLI-bundle behavior was documented but ASPIRE010 was not.

  • Modified: src/frontend/src/content/docs/get-started/aspire-sdk.mdx

Note

This draft PR needs human review before merging.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants