Remove ADP.IsEmpty()

[ErikEJ]

No description provided.

[ErikEJ]

@samsharma2700@mdaigle Could some start the CI?

[mdaigle]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[paulmedynski]

Removing stuff like this is very satisfying :)

[apoorvdeshmukh]

[codecov]

Codecov Report

Attention: Patch coverage is 73.46939% with 13 lines in your changes missing coverage. Please review.

Project coverage is 66.26%. Comparing base (aa35edc) to head (ad0b284).
Report is 4 commits behind head on main.

Files with missing lines	Patch %	Lines
...x/src/Microsoft/Data/SqlClient/SqlDataReaderSmi.cs	0.00%	6 Missing ⚠️
...fx/src/Microsoft/Data/Common/DBConnectionString.cs	25.00%	3 Missing ⚠️
...x/src/Microsoft/Data/Common/DbConnectionOptions.cs	80.00%	1 Missing ⚠️
...rc/Microsoft/Data/SqlClient/SqlClientPermission.cs	0.00%	1 Missing ⚠️
...etfx/src/Microsoft/Data/SqlClient/SqlConnection.cs	87.50%	1 Missing ⚠️
...nt/netfx/src/Microsoft/Data/SqlClient/TdsParser.cs	0.00%	1 Missing ⚠️

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

HEAD has 1 upload less than BASE

Flag	BASE (aa35edc)	HEAD (ad0b284)
addons	1	0

Additional details and impacted files

@@ Coverage Diff @@## main #3277 +/- ##
==========================================
- Coverage 72.91% 66.26% -6.66% 
==========================================
Files 296 289 -7 Lines 57257 56901 -356 ==========================================
- Hits 41751 37704 -4047 - Misses 15506 19197 +3691 

Flag	Coverage Δ
addons	?
netcore	68.96% <100.00%> (-6.15%)	⬇️
netfx	65.26% <73.46%> (-6.19%)	⬇️

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.