Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

System.Text.Json token size limits #26730

Merged
merged 4 commits into from Nov 3, 2021
Merged

System.Text.Json token size limits #26730

merged 4 commits into from Nov 3, 2021

Conversation

tdykstra
Copy link
Contributor

@tdykstra tdykstra commented Oct 29, 2021 •
edited

Fixes #25839
Fixes #25647

@dotnet-bot dotnet-bot added this to the October 2021 milestone Oct 29, 2021
@tdykstra tdykstra requested review from adegeo and a team October 29, 2021 17:02
@tdykstra tdykstra added this to In progress in November 2021 via automation Nov 1, 2021
@tdykstra tdykstra requested review from IEvangelist and removed request for adegeo November 1, 2021 18:03
IEvangelist
IEvangelist approved these changes Nov 3, 2021
View reviewed changes
Copy link
Member

@IEvangelist IEvangelist left a comment

Choose a reason for hiding this comment

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

Looks good to me, :shipit: when you're ready.

@tdykstra tdykstra merged commit 04ec7f0 into dotnet:main Nov 3, 2021
November 2021 automation moved this from In progress to Done Nov 3, 2021
@tdykstra tdykstra deleted the stjconstants branch November 3, 2021 16:57
@tdykstra tdykstra added the okr-freshness OKR: Freshness of content label Apr 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@IEvangelist IEvangelist IEvangelist approved these changes

Assignees
No one assigned
Labels
dotnet-fundamentals/svc okr-freshness OKR: Freshness of content
Projects
No open projects
November 2021
  
Done
Milestone
October 2021
Development

Successfully merging this pull request may close these issues.

Include size limitations on migration page Newtonsoft.Json has already supported MaxDepth
3 participants
@tdykstra @IEvangelist @dotnet-bot