Skip to content

Conversation

cheenamalhotra
Copy link
Member

@cheenamalhotra cheenamalhotra commented Jun 6, 2025

Ports #3391 to release/6.0

Docs can be found here: https://github.com/dotnet/sqlclient-api-docs/pull/83

* Update Docs - fix lists and tables

* Fixes

* Fix typos
@cheenamalhotra cheenamalhotra marked this pull request as ready for review June 6, 2025 00:01
@Copilot Copilot AI review requested due to automatic review settings June 6, 2025 00:01
Copy link
Contributor

@Copilot 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

Ports documentation updates from #3391 to the 6.0 release branch, standardizing XML list and table markup across reference snippets.

  • Wraps all <item> contents in <description> tags for consistency in bullet and numbered lists
  • Adjusts nested <term>/<description> elements in table lists and nested lists
  • Updates header and item elements in snippets under both Microsoft.SqlServer.Server and Microsoft.Data.SqlClient

Reviewed Changes

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

Show a summary per file
File Description
doc/snippets/Microsoft.SqlServer.Server/SqlUserDefinedTypeAttribute.xml Wrapped bullet <item> content in <description> tags
doc/snippets/Microsoft.SqlServer.Server/Format.xml Wrapped bullet <item> content in <description> tags
doc/snippets/Microsoft.Data.SqlClient/SqlTransaction.xml Wrapped bullet <item> content in <description> tags
doc/snippets/Microsoft.Data.SqlClient/SqlRowUpdatingEventArgs.xml Wrapped numbered <item> content in <description> tags
doc/snippets/Microsoft.Data.SqlClient/SqlRowUpdatedEventArgs.xml Wrapped numbered <item> content in <description> tags
doc/snippets/Microsoft.Data.SqlClient/SqlParameter.xml Wrapped bullet <item> content in <description> tags
doc/snippets/Microsoft.Data.SqlClient/SqlDependency.xml Wrapped bullet <item> content in <description> tags
doc/snippets/Microsoft.Data.SqlClient/SqlDataReader.xml Wrapped bullet <item> content in <description> tags
doc/snippets/Microsoft.Data.SqlClient/SqlDataAdapter.xml Wrapped numbered <item> content in <description> tags
doc/snippets/Microsoft.Data.SqlClient/SqlCredential.xml Wrapped bullet <item> content in <description> tags
doc/snippets/Microsoft.Data.SqlClient/SqlConnectionStringBuilder.xml Wrapped bullet <item> content; adjusted table markup
doc/snippets/Microsoft.Data.SqlClient/SqlConnection.xml Wrapped bullet <item> content in <description> tags
doc/snippets/Microsoft.Data.SqlClient/SqlConfigurableRetryFactory.xml Wrapped bullet <item> content in <description> tags
doc/snippets/Microsoft.Data.SqlClient/SqlCommandColumnEncryptionSetting.xml Wrapped bullet <item> content in <description> tags
doc/snippets/Microsoft.Data.SqlClient/SqlCommandBuilder.xml Wrapped bullet <item> content in <description> tags
doc/snippets/Microsoft.Data.SqlClient/SqlBulkCopy.xml Wrapped bullet <item> content in <description> tags
doc/snippets/Microsoft.Data.SqlClient/ActiveDirectoryAuthenticationProvider.xml Wrapped bullet <item> content in <description> tags
Comments suppressed due to low confidence (1)

doc/snippets/Microsoft.Data.SqlClient/SqlConnectionStringBuilder.xml:2009

  • In a list type="table" header, use <term> tags for each column heading rather than <description>. Replace <description>Type</description>, <description>Information provided</description>, and <description>Return value</description> with <term>Type</term>, <term>Information provided</term>, and <term>Return value</term> respectively.
<description>Type</description>

@cheenamalhotra cheenamalhotra changed the title Update Docs - fix lists and tables (#3391) [6.0] Update Docs - fix lists and tables (#3391) Jun 9, 2025
@cheenamalhotra cheenamalhotra merged commit 392002e into release/6.0 Jun 10, 2025
6 checks passed
@cheenamalhotra cheenamalhotra deleted the dev/cheena/fix-api-docs-6.0 branch June 10, 2025 16:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants