Skip to content

Chore: update repository to modern slnx format#338

Open
JanHyka wants to merge 6 commits into
aws:devfrom
JanHyka:janhyka/chore_slnx
Open

Chore: update repository to modern slnx format#338
JanHyka wants to merge 6 commits into
aws:devfrom
JanHyka:janhyka/chore_slnx

Conversation

@JanHyka

@JanHyka JanHyka commented May 11, 2026

Copy link
Copy Markdown
Contributor

Issue #, if available:
n/a

Description of changes:

To prevent complicated merges and modifications of older .sln format caused by edits in different IDEs, migrate to slnx (doc: https://devblogs.microsoft.com/visualstudio/new-simpler-solution-file-format/ )

  1. Migrate solution to modern, IDE agnostic slnx format by executing dotnet sln migrate.
  2. Update explicit file names where found in build pipeline.

Pls execute all pipelines to validate functionality.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@JanHyka JanHyka requested review from a team as code owners May 11, 2026 10:37
@JanHyka JanHyka requested review from normj and philasmar May 11, 2026 10:37

@GarrettBeatty GarrettBeatty left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

thanks for this PR!

@GarrettBeatty GarrettBeatty added the Release Not Needed Add this label if a PR does not need to be released. label May 11, 2026
@GarrettBeatty

Copy link
Copy Markdown
Contributor

The slnx solution format requires .NET SDK 9.0.200+. The CI was
using 8.x which doesn't recognize the <Solution> element.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@GarrettBeatty

Copy link
Copy Markdown
Contributor

@GarrettBeatty GarrettBeatty removed the Release Not Needed Add this label if a PR does not need to be released. label May 11, 2026
@GarrettBeatty

Copy link
Copy Markdown
Contributor

@@ -0,0 +1,11 @@
{

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

we don't need a change file for this change. We are not introducing a customer facing change.

@GarrettBeatty GarrettBeatty added the Release Not Needed Add this label if a PR does not need to be released. label May 12, 2026
@GarrettBeatty GarrettBeatty requested a review from philasmar May 12, 2026 18:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Release Not Needed Add this label if a PR does not need to be released.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants