Add missing package reference to System.Data.Common

[cheenamalhotra]

Fixes #4062

[Copilot]

Pull request overview

Adds an explicit System.Data.Common dependency to the SqlClient build and packaging metadata so the net462 target has a direct/declared reference instead of relying on implicit or transitive resolution.

Changes:

• Add System.Data.Common as a PackageReference for the net462 build of Microsoft.Data.SqlClient.
• Add System.Data.Common to the netfx reference-assembly project’s package references.
• Pin System.Data.Common centrally and declare it as a net462 dependency in the .nuspec.

Reviewed changes

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

File	Description
tools/specs/Microsoft.Data.SqlClient.nuspec	Declares System.Data.Common (4.3.0) as a net462 NuGet dependency.
src/Microsoft.Data.SqlClient/src/Microsoft.Data.SqlClient.csproj	Adds System.Data.Common package reference for the net462 target.
src/Microsoft.Data.SqlClient/netfx/ref/Microsoft.Data.SqlClient.csproj	Adds System.Data.Common package reference for the net462 reference assembly build.
Directory.Packages.props	Centrally pins System.Data.Common to 4.3.0.

---

You can also share your feedback on Copilot code review. Take the survey.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 68.44%. Comparing base (ce38e64) to head (e856b24).
⚠️ Report is 1 commits behind head on main.

❗ There is a different number of reports uploaded between BASE (ce38e64) and HEAD (e856b24). Click for more details.

HEAD has 1 upload less than BASE

Flag	BASE (ce38e64)	HEAD (e856b24)
CI-SqlClient	1	0

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #4063      +/-   ##
==========================================
- Coverage   75.09%   68.44%   -6.66%     
==========================================
  Files         280      275       -5     
  Lines       43830    66930   +23100     
==========================================
+ Hits        32914    45807   +12893     
- Misses      10916    21123   +10207     

Flag	Coverage Δ
CI-SqlClient	?
PR-SqlClient-Project	68.44% <ø> (?)

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.

[mdaigle]

@copilot When were these references removed? Can you point to a specific PR?

[Copilot]

@mdaigle I've opened a new pull request, #4066, to work on those changes. Once the pull request is ready, I'll request review from you.