Skip to content

Migrate to slnx solution file format#4070

Merged
paulmedynski merged 2 commits intomainfrom
dev/paul/slnx
Mar 20, 2026
Merged

Migrate to slnx solution file format#4070
paulmedynski merged 2 commits intomainfrom
dev/paul/slnx

Conversation

@paulmedynski
Copy link
Contributor

Description

I used Opus 4.6 to plan a migration from the legacy sln format to the modern slnx format, and it didn't find any legitimate issues. I encourage folks to try out the new slnx file on this branch with their IDEs and tooling.

Benefits include:

  • slnx is the future, so we should embrace it.
  • Much simpler diffs in PRs.
  • No more GUID overload.

Testing

Our pipelines don't use the solution file, but they can check for any other regressions.

@paulmedynski paulmedynski added this to the 7.1.0-preview1 milestone Mar 20, 2026
@paulmedynski paulmedynski requested a review from a team as a code owner March 20, 2026 14:56
Copilot AI review requested due to automatic review settings March 20, 2026 14:56
@paulmedynski paulmedynski added the Area\Engineering Use this for issues that are targeted for changes in the 'eng' folder or build systems. label Mar 20, 2026
@github-project-automation github-project-automation bot moved this to To triage in SqlClient Board Mar 20, 2026
Copy link
Contributor

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 migrates the repository’s primary developer solution from the legacy Visual Studio .sln format to the newer XML-based .slnx format under src/, aiming to simplify solution diffs and reduce GUID-heavy churn.

Changes:

  • Added src/Microsoft.Data.SqlClient.slnx as the solution definition in .slnx format.
  • Removed the legacy src/Microsoft.Data.SqlClient.sln.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
src/Microsoft.Data.SqlClient.slnx New XML-based solution file containing projects, folders, and solution items.
src/Microsoft.Data.SqlClient.sln Deleted the legacy .sln solution previously used by IDEs/tooling.

Copy link
Contributor

@benrr101 benrr101 left a comment

Choose a reason for hiding this comment

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

This is probably a good change :) I'm tired of the guid changes everywhere. Though I hope we can come up with a better folder scheme soonish.

@codecov
Copy link

codecov bot commented Mar 20, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 68.08%. Comparing base (07a9280) to head (81761f7).
⚠️ Report is 2 commits behind head on main.

❗ There is a different number of reports uploaded between BASE (07a9280) and HEAD (81761f7). Click for more details.

HEAD has 2 uploads less than BASE
Flag BASE (07a9280) HEAD (81761f7)
CI-SqlClient 2 0
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4070      +/-   ##
==========================================
- Coverage   75.17%   68.08%   -7.10%     
==========================================
  Files         280      275       -5     
  Lines       43830    66924   +23094     
==========================================
+ Hits        32951    45563   +12612     
- Misses      10879    21361   +10482     
Flag Coverage Δ
CI-SqlClient ?
PR-SqlClient-Project 68.08% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@paulmedynski paulmedynski merged commit 40db478 into main Mar 20, 2026
299 checks passed
@paulmedynski paulmedynski deleted the dev/paul/slnx branch March 20, 2026 18:44
@github-project-automation github-project-automation bot moved this from To triage to Done in SqlClient Board Mar 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area\Engineering Use this for issues that are targeted for changes in the 'eng' folder or build systems.

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants