Update Orleans and Aspire integration documentation - #54201
Conversation
.NET Aspire -> Aspire
There was a problem hiding this comment.
Pull request overview
Note
Copilot was unable to run its full agentic suite in this review.
Updates the Orleans hosting documentation to reflect “Aspire” (vs “.NET Aspire”) terminology and refreshes related setup links.
Changes:
- Renamed headings/front-matter to “Orleans and Aspire integration”.
- Updated external/reference links (aspire.dev, VS Code extension link).
- Adjusted wording in pivoted sections to use “Aspire” terminology.
|
@Meir017, I agree with the Copilot suggestions. Here are the issues and fixes: Issue 1: Branding inconsistency
Fix: Keep ".NET Aspire" as the primary name throughout the document. If shorthand is needed, add "(Aspire)" after first mention, e.g., ".NET Aspire (hereafter referred to as Aspire)". Issue 2: External links vs internal links Fix: Restore the Microsoft Learn links for consistency, localization, and long-term stability. Use aspire.dev only for content that doesn't exist on Microsoft Learn. Issue 3: Capitalization Issue 4: Mixed terminology in pivot section Summary of changes needed: | Title | "Orleans and Aspire integration" | "Orleans and .NET Aspire integration" | I can help create a PR with these fixes if needed. |
|
The official name is Aspire, not .NET Aspire |
|
@copilot resolve the merge conflicts in this pull request |
Co-authored-by: Genevieve Warren <24882762+gewarren@users.noreply.github.com>
|
Closing and reopening pr quickly, to get the build system and checks to give a fresh look. Three check systems were blocked. |
|
It looks like there remains a merge conflict address, (which in turn may be holding up the Policheck and OPS status checker): Selecting "Resolve conflicts > Edit on the web" below on this PR page will show three small items to resolve one way or the other if you want to make the decision manually. |
.NET Aspire -> Aspire
Summary
FYI @IEvangelist @davidfowl