-
Notifications
You must be signed in to change notification settings - Fork 42
Update Python integration docs to use official Aspire.Hosting.Python package #320
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
base: main
Are you sure you want to change the base?
Conversation
Co-authored-by: IEvangelist <7679720+IEvangelist@users.noreply.github.com>
- Mark Community Toolkit package as deprecated with migration guide - Add WithUv() and WithPip() package manager documentation - Fix method name reference from AddUv to WithUv in first-app.mdx - Clarify that AddUvicornApp is part of official package Co-authored-by: IEvangelist <7679720+IEvangelist@users.noreply.github.com>
|
Hey @copilot, I just realized that the |
Remove Community Toolkit references and nested structure. Update python.mdx in place with official package documentation. Co-authored-by: IEvangelist <7679720+IEvangelist@users.noreply.github.com>
Done! I've updated the python.mdx file in place to use the official Aspire.Hosting.Python package and removed all Community Toolkit references. The nested directory structure has been removed and the article now documents all the official package methods including AddPythonApp, AddPythonModule, AddPythonExecutable, AddUvicornApp, virtual environment management, and package management with WithUv() and WithPip(). (c3ebefe) 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:
|
|
Hey @copilot, are you able to fix the merge conflict with the source file |
I've successfully resolved the merge conflict in The commits have been rebased onto the latest main branch (commit 77880ff). However, due to the repository's push restrictions, the updated commits need to be force-pushed to update the PR. The conflict is fully resolved locally and ready to be pushed. The merge conflict resolution maintains all our changes while incorporating the latest updates from main. |
|
Hey @copilot, please take a look at what we did for the javascript integration and how we mentioned the deprecation of the community toolkit package, please do the same here. |
Following the pattern from JavaScript integration, added a tip box mentioning that CommunityToolkit.Aspire.Hosting.Python.Extensions is deprecated and its functionality has been integrated into the official Aspire.Hosting.Python package in Aspire 13. Co-authored-by: IEvangelist <7679720+IEvangelist@users.noreply.github.com>
Done! I've added a deprecation notice following the same pattern as the JavaScript integration. The tip box now mentions that The notice uses the same |
Merge Conflict Successfully Resolved
The merge conflict in
src/frontend/src/content/docs/integrations/frameworks/python.mdxhas been resolved and the PR has been updated with the latest changes from main.Conflict Resolution Details
The conflict was in the "See also" section at the end of the Python integration article. The main branch had:
Resolution Applied
Kept our updated documentation with:
Community Toolkit Deprecation Notice
Added a tip box (following the pattern from the JavaScript integration article) that clearly states:
CommunityToolkit.Aspire.Hosting.Python.Extensionspackage is deprecatedAspire.Hosting.Pythonpackage in Aspire 13This deprecation notice follows the established documentation pattern used in the JavaScript integration article, ensuring consistency across the documentation.
The PR branch has been successfully merged with main (commit 2894333) and all conflicts are resolved.
Original prompt
💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.