Skip to content

Conversation

@alesma
Copy link

@alesma alesma commented Dec 9, 2025

Removed the option to use JSON format for storage.

This option was removed in

316ddd7

Microsoft Reviewers: Open in CodeFlow

Removed the option to use JSON format for storage.

This option was removed in 

dotnet@316ddd7
Copilot AI review requested due to automatic review settings December 9, 2025 08:54
Copy link
Contributor

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

This PR removes an outdated configuration option from the ADO.NET persistence documentation. The UseJsonFormat option was previously removed from the codebase in commit 316ddd7, but the documentation example was not updated at that time.

  • Removes the UseJsonFormat configuration line from the README example

@ReubenBond
Copy link
Member

@benjaminpetit ptal

@ranma42
Copy link

ranma42 commented Dec 11, 2025

A similar leftover (in the documentation) is here:

/// Optional configuration params:
/// <c>AdoInvariant</c> -- defaults to <c>Microsoft.Data.SqlClient</c>
/// <c>UseJsonFormat</c> -- defaults to <c>false</c>
/// <c>UseXmlFormat</c> -- defaults to <c>false</c>
/// <c>UseBinaryFormat</c> -- defaults to <c>true</c>
/// </para>

@alesma
Copy link
Author

alesma commented Dec 11, 2025

@dotnet-policy-service agree

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.

4 participants