Skip to content

Conversation

davidfowl
Copy link
Member

…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.

…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.
@Copilot Copilot AI review requested due to automatic review settings May 13, 2025 23:17
Copy link
Contributor

@Copilot Copilot AI left a 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 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 github-actions bot added the area-engineering-systems infrastructure helix infra engineering repo stuff label May 13, 2025
"remote.otherPortsAttributes": {
"onAutoForward": "ignore"
},
"dotnet.defaultSolution": "Aspire.sln"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

@davidfowl davidfowl merged commit eefa50f into main May 14, 2025
176 checks passed
@davidfowl davidfowl deleted the davidfowl/move-to-slnx branch May 14, 2025 01:36
@github-actions github-actions bot locked and limited conversation to collaborators Jun 13, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-engineering-systems infrastructure helix infra engineering repo stuff
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants