-
Notifications
You must be signed in to change notification settings - Fork 647
Update diagnostic URLs #9179
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update diagnostic URLs #9179
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Assuming we get the aka links up before we ship, lgtm.
@@ -54,7 +54,7 @@ public static class ContainerAppExtensions | |||
/// </code> | |||
/// </example> | |||
/// </remarks> | |||
[Experimental("ASPIREACADOMAINS001", UrlFormat = "https://aka.ms/dotnet/aspire/diagnostics#{0}")] | |||
[Experimental("ASPIREACADOMAINS001", UrlFormat = "https://aka.ms/aspire/diagnostic/{0}")] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
if it matters, some of these are not in https://github.com/dotnet/aspire/blob/main/docs/list-of-diagnostics.md
(not sure why we have htat file exactly though)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It could be a reference file devs should use to look up existing IDs? But honestly you should have a review process about creating those with some approval mechanism and really you should switch to using a constant. It's very easy to accidentally name one wrong when you're trying to apply it. 😁
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Any reason we're changing the convention (which is https://aka.ms/dotnet/<repo>/diagnostics#{0}
)?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, we're doing work (such as the CLI) to make Aspire accessible to non-.NET developers, so it makes sense.
I created the aka links using diagnostics not diagnostic... Let me fix. We can alter the code or adjust each link. Find replace in code is quicker than using the web UI to adjust each link. |
Description
Technically depends on dotnet/docs-aspire#3379 but it's not required to merge. Docs will ship soon.DoneIntellisense didn't show
HelpLink
on the MSBuild actions, but the MSBuild docs said it's supported.Fixes #8936
Checklist