-
Notifications
You must be signed in to change notification settings - Fork 14
Add how-to guide for adopting Aspire in existing applications #67
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
Conversation
Co-authored-by: IEvangelist <7679720+IEvangelist@users.noreply.github.com>
Co-authored-by: IEvangelist <7679720+IEvangelist@users.noreply.github.com>
IEvangelist
left a comment
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.
Hey @copilot, please address feedback.
src/frontend/src/content/docs/get-started/add-aspire-existing-app.mdx
Outdated
Show resolved
Hide resolved
src/frontend/src/content/docs/get-started/add-aspire-existing-app.mdx
Outdated
Show resolved
Hide resolved
…Host, add config injection explanation Co-authored-by: IEvangelist <7679720+IEvangelist@users.noreply.github.com>
All feedback addressed in commit 11c311f:
Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
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.
Pull Request Overview
This PR adds a comprehensive how-to guide for adopting Aspire orchestration in existing applications. The guide demonstrates using aspire init to incrementally add Aspire to C#, Python, and JavaScript applications without requiring a full rewrite.
Key changes:
- New documentation file explaining the gradual adoption workflow using file-based AppHosts
- Updated InstallPackage component to support omitting PackageReference tab for certain scenarios
- Added sidebar navigation entry with internationalized translations across 15 languages
Reviewed Changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| src/frontend/src/content/docs/get-started/add-aspire-existing-app.mdx | Complete how-to guide covering aspire init, polyglot orchestration, telemetry setup, and integrations |
| src/frontend/src/components/InstallPackage.astro | Added optional omitPackageReference prop to conditionally hide PackageReference tab |
| src/frontend/sidebar.topics.ts | Added sidebar entry with translations for the new guide |
| src/frontend/src/data/aspire-integrations.json | Updated NuGet package download counts (automated update) |
Comments suppressed due to low confidence (1)
src/frontend/src/components/InstallPackage.astro:31
- The title variable uses
AppHost.cs(uppercase) but the tab label on line 54 usesapphost.cs(lowercase). For consistency with file-based AppHosts, the title should beC# — apphost.csto match the lowercase convention used throughout the codebase for file-based AppHosts.
const appHostTitle = `C# — AppHost.cs`;
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
@copilot apply changes based on the comments in this thread |
…try example Co-authored-by: IEvangelist <7679720+IEvangelist@users.noreply.github.com>
|
Just as a heads up, I was blocked by some firewall rules while working on your feedback. Expand below for details. Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
Co-authored-by: IEvangelist <7679720+IEvangelist@users.noreply.github.com>
Added the "How-to" badge back to the sidebar navigation with variant "note" and translations for all 15 languages in commit fbac9b8. Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
src/frontend/src/content/docs/get-started/add-aspire-existing-app.mdx
Outdated
Show resolved
Hide resolved
Meir017
left a comment
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.
Minor comments here
…app.mdx Co-authored-by: Meir Blachman <meirblachman@gmail.com>
|
@IEvangelist It might be better to have a single location that describes the requirements for aspire (.NET version and IDEs) abd then reference it, instead of having minimal instructions in multiple pages |
You're right! This is what we did on Learn, where we had a Setup and tooling doc with environment selector. Seems similar to what @mitchdenny was suggesting here too in #64. |


Add how-to guide for adopting Aspire in existing applications
✅ All Feedback Addressed
Addresses #60 by documenting the gradual adoption path for adding Aspire orchestration to existing .NET applications using
aspire init.Changes
New how-to guide
/get-started/add-aspire-existing-app/covering the incremental adoption workflowapphost.cs) created byaspire initaspire init→ ServiceDefaults → integrationsKey sections
aspire initinteractive mode walkthrough for creating file-based AppHostaspire runand dashboard explorationLatest fixes
PeriodicExportingMetricReaderimport to JavaScript telemetry example (commit d9bfcd4)Configuration injection
Explains how Aspire's AppHost automatically injects configuration from hosting integrations into dependent resources:
These are passed through .NET's standard configuration system, eliminating manual connection string management.
Example: Adding Redis integration
Navigation
The guide emphasizes minimal code changes and non-invasive integration, addressing the OP's request for "where to start from" and "how to do this gradually."
Original prompt
💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.