Azure Split - Step 2 - New Files

[paulmedynski]

Description

This PR contains the new Abstractions and Azure package files:

• Added the new Abstractions and Azure source files and associated pipeline files.
• Setup build.proj targets and default version numbers.
• Standardized project CLS compliance.

PR series:

• Azure Split - Step 1 - Prep Work #3902:
  • Merge changes that aren't directly related to the Azure split work, but that setup the stage for that work.
• Azure Split - Step 2 - New Files #3904:
  • Merge new files and related changes that do not affect the existing MDS projects or artifacts.
• Azure Split - Step 3 - Tie Everything Together #3908:
  • Merge the changes to MDS and pipelines that tie everything together.
• Azure Split - Step 4 - Cleanup #3917:
  • Any remaining tasks or cleanup.

Testing

• Normal PR/CI pipeline runs will validate most changes.
• Manual inspection of pipeline logs and artifacts will confirm some of the fiddly parts.

[Copilot]

Pull request overview

This pull request is Step 2 of the Azure split effort, adding new Abstractions and Azure extension packages to decouple Azure dependencies from the main Microsoft.Data.SqlClient package. The PR introduces two new packages with their source code, tests, documentation, and CI/CD pipeline configurations while standardizing CLS compliance across projects.

Changes:

• Added new Microsoft.Data.SqlClient.Extensions.Abstractions package with base types for authentication and related abstractions
• Added new Microsoft.Data.SqlClient.Extensions.Azure package containing Azure authentication implementations moved from MDS
• Standardized CLS compliance by using AssemblyAttribute declarations instead of embedded assembly attributes
• Added comprehensive CI/CD pipeline support for building and testing both new packages across multiple platforms

Reviewed changes

Copilot reviewed 69 out of 69 changed files in this pull request and generated 13 comments.

Show a summary per file

File	Description
tools/targets/GenerateThisAssemblyCs.targets	Simplified to unconditionally generate ThisAssembly.cs with configurable namespace
tools/props/Versions.props	Added imports for Abstractions and Azure version properties
src/Microsoft.Data.SqlClient/tests/tools/TDS/**/*.csproj	Removed ClsCompliant property (standardization)
src/Microsoft.Data.SqlClient/**/Microsoft.Data.SqlClient.csproj	Added CLS compliance via AssemblyAttribute
src/Microsoft.Data.SqlClient.sln	Added folder structure and projects for Abstractions and Azure packages
src/Microsoft.Data.SqlClient.Extensions/Abstractions/**	Complete Abstractions package with source, tests, and documentation
src/Microsoft.Data.SqlClient.Extensions/Azure/**	Complete Azure package with source, tests, and documentation
eng/pipelines/**	New CI/CD stages, jobs, and variables for building/testing new packages
build.proj	Added targets for building Abstractions and Azure packages
Directory.Packages.props	Added package version management for new packages
BUILDGUIDE.md	Updated documentation with new build targets

[Copilot]

Pull request overview

Copilot reviewed 72 out of 72 changed files in this pull request and generated no new comments.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 67.52%. Comparing base (88de2b9) to head (74587f9).
⚠️ Report is 4 commits behind head on main.

❗ There is a different number of reports uploaded between BASE (88de2b9) and HEAD (74587f9). Click for more details.

HEAD has 1 upload less than BASE

Flag	BASE (88de2b9)	HEAD (74587f9)
addons	1	0

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #3904      +/-   ##
==========================================
- Coverage   74.93%   67.52%   -7.41%     
==========================================
  Files         269      263       -6     
  Lines       43247    66191   +22944     
==========================================
+ Hits        32407    44696   +12289     
- Misses      10840    21495   +10655     

Flag	Coverage Δ
addons	?
netcore	67.50% <ø> (-7.49%)	⬇️
netfx	66.54% <ø> (-7.62%)	⬇️

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.

[Copilot]

Pull request overview

Copilot reviewed 72 out of 72 changed files in this pull request and generated 3 comments.

[Copilot]

Pull request overview

Copilot reviewed 72 out of 72 changed files in this pull request and generated 8 comments.

[Copilot]

Pull request overview

Copilot reviewed 72 out of 72 changed files in this pull request and generated 5 comments.

[mdaigle]

One required change. Others are suggestions for future.

[benrr101]

Most of my comments are probably unactionable today, but I'd at least like to see some todo's added to address them later.

[Copilot]

Pull request overview

Copilot reviewed 72 out of 72 changed files in this pull request and generated 2 comments.