Skip to content

Switch solution to .slnx - #521

Merged
danielchalmers merged 2 commits into
devfrom
claude/switch-to-slnx
Jul 27, 2026
Merged

Switch solution to .slnx#521
danielchalmers merged 2 commits into
devfrom
claude/switch-to-slnx

Conversation

@danielchalmers

@danielchalmers danielchalmers commented Jul 27, 2026

Copy link
Copy Markdown
Member

build from the XML .slnx solution format like our other repos.

Changes

  • Generated src/MudBlazor.Templates.slnx with dotnet sln src/MudBlazor.Templates.sln migrate, then deleted the old .sln.
  • Repointed the four dotnet invocations in continuous.yml and release.yml.

The resulting file is the whole solution, and matches the shape used in the other two repos:

<Solution>
  <Project Path="MudBlazor.Templates.csproj" />
</Solution>

src/mudblazor/MudBlazor.Template.sln stays a .sln. It is template content, not build infrastructure — .template.config/template.json references it by name as a primary output and in two exclude/rename rules, and it ships to users through dotnet new. Converting it would change generated project output and impose an SDK floor (.slnx needs 9.0.200+) on everyone using the template. That is a product decision, separate from the format this repo builds with, so it is out of scope here.

For the same reason the *.sln text eol=crlf rule in .gitattributes is unchanged — it still applies to that file.

Matches MudBlazor/ThemeManager and MudBlazor/MudBlazor.Icons, which both use
the XML `.slnx` solution format.

Generated with `dotnet sln src/MudBlazor.Templates.sln migrate`, then removed
the old `.sln` and repointed the `dotnet` invocations in `continuous.yml` and
`release.yml`.

Note that `src/mudblazor/MudBlazor.Template.sln` is deliberately left alone.
That file is template content — it is referenced by
`.template.config/template.json` and ships to users through `dotnet new`, so
converting it would change generated output and impose an SDK floor on
consumers. That is a separate decision from the format this repo builds with.
@danielchalmers
danielchalmers merged commit f59663f into dev Jul 27, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant