Skip to content

Clarify baseConnectionString in connection string builder example - #55010

Merged
adegeo merged 2 commits into
mainfrom
copilot/fix-connection-string-builder-error-again
Jul 24, 2026
Merged

Clarify baseConnectionString in connection string builder example#55010
adegeo merged 2 commits into
mainfrom
copilot/fix-connection-string-builder-error-again

Conversation

Copilot AI commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

The Connection string builder section showed a code snippet using baseConnectionString, but its definition was outside the snippet region and invisible to readers, causing a confusing ArgumentException when users passed a bare file path instead of a valid connection string.

Changes

  • docs/standard/data/sqlite/connection-strings.md: Added a [!NOTE] callout before the code snippet clarifying that baseConnectionString is an existing connection string, such as Data Source=EncryptionSample.db.

Internal previews

📄 File 🔗 Preview link
docs/standard/data/sqlite/connection-strings.md docs/standard/data/sqlite/connection-strings

Copilot AI review requested due to automatic review settings July 23, 2026 23:14

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because there is no eligible user to bill. To allow Copilot reviews on bot-authored pull requests, enable direct organization billing in your organization's Copilot settings.

Copilot AI linked an issue Jul 23, 2026 that may be closed by this pull request
Copilot AI changed the title [WIP] Fix connection string builder code error Clarify baseConnectionString in connection string builder example Jul 23, 2026
Copilot AI requested a review from adegeo July 23, 2026 23:17
@adegeo
adegeo marked this pull request as ready for review July 24, 2026 17:02
@adegeo
adegeo requested a review from a team as a code owner July 24, 2026 17:02
@adegeo
adegeo enabled auto-merge (squash) July 24, 2026 17:04
@adegeo
adegeo merged commit 6e01f76 into main Jul 24, 2026
12 checks passed
@adegeo
adegeo deleted the copilot/fix-connection-string-builder-error-again branch July 24, 2026 17:12
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.

Connection string builder code error

4 participants