Refactor solution file from .sln to .slnx format and update related d…#9295
Merged
Refactor solution file from .sln to .slnx format and update related d…#9295
Conversation
…ocumentation and scripts - Created new Aspire.slnx file with updated project structure. - Updated contributing and usage documentation to reference Aspire.slnx. - Modified build scripts to use Aspire.slnx instead of Aspire.sln. - Adjusted project template configuration to exclude both .sln and .slnx files. - Updated startvs.cmd to default to Aspire.slnx when no solution is specified.
Contributor
There was a problem hiding this comment.
Pull Request Overview
This PR refactors the solution file references from Aspire.sln to Aspire.slnx, ensuring consistency across documentation, build scripts, project templates, and configuration files.
- Updated startvs.cmd, build scripts, and workflow files to reference Aspire.slnx.
- Modified project templates and documentation to exclude or use Aspire.slnx.
- Adjusted .gitattributes and devcontainer configuration for the new solution file format.
Reviewed Changes
Copilot reviewed 11 out of 11 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| startvs.cmd | Updated default solution reference to Aspire.slnx. |
| src/Aspire.ProjectTemplates/templates/aspire-empty/.template.config/template.json | Added exclusion for *.slnx alongside *.sln in project templates. |
| eng/common/build.ps1 | Updated help message to reflect extension change. |
| eng/build.ps1 | Set the solution path to Aspire.slnx. |
| docs/using-latest-daily.md | Modified documentation to refer to Aspire.slnx. |
| docs/contributing.md | Changed instructions to open Aspire.slnx. |
| .github/workflows/update-dependencies.yml | Adjusted dependency update command to use Aspire.slnx. |
| .gitattributes | Added file pattern for *.slnx. |
| .devcontainer/devcontainer.json | Updated default solution setting to Aspire.slnx. |
github-actions
bot
added
the
area-engineering-systems
| "remote.otherPortsAttributes": { | ||
| "onAutoForward": "ignore" | ||
| }, | ||
| "dotnet.defaultSolution": "Aspire.sln" |
Contributor
There was a problem hiding this comment.
I believe devcontainer will respect the settings in the local .vscode directory so we can just have all the customizations there so they work for non-DevContainer users as well.
captainsafia
approved these changes
May 14, 2025
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
…ocumentation and scripts