Skip to content

Fix API docs: SqlVector generic type refs, SqlDataAdapter event xrefs, and retry xref encoding#4093

Draft
Copilot wants to merge 2 commits intomainfrom
copilot/sub-pr-4084
Draft

Fix API docs: SqlVector generic type refs, SqlDataAdapter event xrefs, and retry xref encoding#4093
Copilot wants to merge 2 commits intomainfrom
copilot/sub-pr-4084

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 26, 2026

Description

Fixes broken API doc cross-references observed in dotnet/sqlclient-api-docs#94.

  • SqlDataReader.xml — Fix GetSqlVector <see cref> targets from non-generic SqlVector to SqlVector\1`
  • SqlDataAdapter.xml — Replace <see cref="M:...OnRowUpdating/OnRowUpdated"> (protected method refs) with <see cref="E:...RowUpdating/RowUpdated"> (event refs)
  • SqlCommand.xml — Fix URL-encoded generic xrefs (%60%601) to DocFX backtick syntax ( 1 / 0 ) for Execute/ExecuteAsync on SqlRetryLogicBaseProvider; fix SqlVector%2ASqlVector\1`; trailing-whitespace cleanup in embedded code samples

Issues

https://github.com/dotnet/sqlclient-api-docs/pull/94

Testing

Changes validated in dotnet/sqlclient-api-docs#94 — doc generation produces correct links with no broken xrefs.

Guidelines

Please review the contribution guidelines before submitting a pull request:


📍 Connect Copilot coding agent with Jira, Azure Boards or Linear to delegate work to Copilot in one click without leaving your project management tool.

@github-project-automation github-project-automation Bot moved this to To triage in SqlClient Board Mar 26, 2026
Copilot AI mentioned this pull request Mar 26, 2026
Copilot AI changed the title [WIP] Fix API docs Fix API docs: SqlVector generic type refs, SqlDataAdapter event xrefs, and retry xref encoding Mar 26, 2026
Copilot AI requested a review from paulmedynski March 26, 2026 14:36
Base automatically changed from dev/ad/fix-docs to main March 27, 2026 20:27
@mdaigle mdaigle moved this from To triage to Backlog in SqlClient Board Apr 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Backlog

Development

Successfully merging this pull request may close these issues.

4 participants