Skip to content

Meta: Migrate solution file from Spark.sln to Spark.slnx - #1420

Merged
kennethmyhra merged 1 commit into
FirelyTeam:masterfrom
losolio:meta/slnx
Sep 17, 2026
Merged

kennethmyhra merged 1 commit into
FirelyTeam:masterfrom
losolio:meta/slnx

Conversation

@losolio

@losolio losolio commented Sep 16, 2026

Copy link
Copy Markdown
Collaborator

Replaces Spark.sln with Spark.slnx, generated with dotnet sln Spark.sln migrate.

@losolio
losolio force-pushed the meta/slnx branch 2 times, most recently from d63d5aa to 421eae0 Compare September 16, 2026 20:42
@losolio
losolio marked this pull request as ready for review September 17, 2026 06:10
Copilot AI balanced review requested due to automatic review settings September 17, 2026 06:10

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟢 Approval recommended

The new Spark.slnx references valid paths and includes the full set of repository .csproj files with no detected omissions.

Pull request overview

This PR migrates the repository’s Visual Studio solution from the legacy Spark.sln format to the XML-based Spark.slnx format (generated via dotnet sln Spark.sln migrate), keeping the same set of projects and solution items while removing the old solution file.

Changes:

  • Added Spark.slnx containing the full solution structure (projects + solution items).
  • Removed the legacy Spark.sln.
File summaries
File Description
Spark.slnx New .slnx solution file containing all projects and solution folders/items.
Spark.sln Removed legacy .sln solution file after migration to .slnx.
Review details
  • Files reviewed: 2/2 changed files
  • Comments generated: 0
  • Review effort level: Lite (auto)

Note

Copilot is running an experiment and ran this review at Lite.


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@kennethmyhra
kennethmyhra merged commit fafd42f into FirelyTeam:master Sep 17, 2026
8 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.

3 participants