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.

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

Review error messages in JSON #72254

Merged
merged 4 commits into from
Jul 15, 2022
Merged

Conversation

krwq
Copy link
Member

@krwq krwq commented Jul 15, 2022

Contributes to: #63686

closes last bullet "Ensure exception messages are still relevant and up-to-date"

@ghost
Copy link

ghost commented Jul 15, 2022

Tagging subscribers to this area: @dotnet/area-system-text-json, @gregsdennis
See info in area-owners.md if you want to be subscribed.

Issue Details

Contributes to: #63686

closes last bullet "Ensure exception messages are still relevant and up-to-date"

Author: krwq
Assignees: -
Labels:

area-System.Text.Json

Milestone: -

Copy link
Member

@eiriktsarpalis eiriktsarpalis left a comment

Choose a reason for hiding this comment

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

Thanks

@krwq
Copy link
Member Author

krwq commented Jul 15, 2022

changed the message per offline conversation

@stephentoub stephentoub merged commit 0fe98fc into dotnet:main Jul 15, 2022
@ghost ghost locked as resolved and limited conversation to collaborators Aug 14, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants