remove redundant Attribute( suffixes

[SimonCropp]

No description provided.

[Copilot]

Pull request overview

This PR standardizes attribute usage by removing redundant Attribute suffixes across several SqlClient types, and includes a few whitespace-only cleanups in comments/preprocessor blocks.

Changes:

• Replaced usages like EditorBrowsableAttribute, ResCategoryAttribute, ResDescriptionAttribute, ConditionalAttribute, and fully-qualified serialization attributes with their shortened forms.
• Cleaned up trailing whitespace and spacing around #endif and comment lines.

Reviewed changes

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

Show a summary per file

File	Description
src/Microsoft.Data.SqlClient/src/Microsoft/Data/SqlClient/SqlParameterCollection.cs	Shortened EditorBrowsableAttribute usage; minor doc comment whitespace cleanup.
src/Microsoft.Data.SqlClient/src/Microsoft/Data/SqlClient/SqlException.cs	Shortened OptionalFieldAttribute / IgnoreDataMember usages (netfx-only) and removed stray whitespace.
src/Microsoft.Data.SqlClient/src/Microsoft/Data/SqlClient/SqlDependency.cs	Shortened HostProtectionAttribute, ResCategoryAttribute, ResDescriptionAttribute; whitespace cleanup.
src/Microsoft.Data.SqlClient/src/Microsoft/Data/SqlClient/SqlDataReader.cs	Shortened EditorBrowsableAttribute usages.
src/Microsoft.Data.SqlClient/src/Microsoft/Data/SqlClient/SqlDataAdapter.cs	Shortened ResCategoryAttribute / ResDescriptionAttribute usages.
src/Microsoft.Data.SqlClient/src/Microsoft/Data/SqlClient/SqlConnection.cs	Shortened ResCategoryAttribute / SettingsBindableAttribute usages; whitespace cleanup.
src/Microsoft.Data.SqlClient/src/Microsoft/Data/SqlClient/SqlCommandBuilder.cs	Shortened EditorBrowsableAttribute, ResCategoryAttribute, ResDescriptionAttribute usages.
src/Microsoft.Data.SqlClient/src/Microsoft/Data/Common/ConnectionString/DbConnectionString.netfx.cs	Shortened ConditionalAttribute usage and whitespace cleanup around #endif.

[paulmedynski]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 2 pipeline(s).

[codecov]

Codecov Report

❌ Patch coverage is 0% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 65.46%. Comparing base (52c7149) to head (96647df).
⚠️ Report is 10 commits behind head on main.

Files with missing lines	Patch %	Lines
...ient/src/Microsoft/Data/SqlClient/SqlConnection.cs	0.00%	1 Missing ⚠️

❗ There is a different number of reports uploaded between BASE (52c7149) and HEAD (96647df). Click for more details.

HEAD has 1 upload less than BASE

Flag	BASE (52c7149)	HEAD (96647df)
CI-SqlClient	1	0

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #4158      +/-   ##
==========================================
- Coverage   74.27%   65.46%   -8.81%     
==========================================
  Files         279      274       -5     
  Lines       42980    66416   +23436     
==========================================
+ Hits        31922    43482   +11560     
- Misses      11058    22934   +11876     

Flag	Coverage Δ
CI-SqlClient	?
PR-SqlClient-Project	65.46% <0.00%> (?)

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.