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

[release/6.0-preview4] Add JSON source generator #51510

Merged

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Apr 19, 2021

Backport of #51300 to release/6.0-preview4

Introduces the System.Text.Json source generator, which helps optimize the performance and size of apps using System.Text.Json.

/cc @layomia

Customer Impact

It is important to us to get early customer feedback on this feature to verify performance & size improvements, as well as test the usability.

Testing

Tests have been added for the generator and it's usage.

Risk

Theses changes are additive and do not modify existing JsonSerializer functionality.

@dotnet-issue-labeler
Copy link

Note regarding the new-api-needs-documentation label:

This serves as a reminder for when your PR is modifying a ref *.cs file and adding/modifying public APIs, to please make sure the API implementation in the src *.cs file is documented with triple slash comments, so the PR reviewers can sign off that change.

@ghost
Copy link

ghost commented Apr 19, 2021

Tagging subscribers to this area: @eiriktsarpalis, @layomia
See info in area-owners.md if you want to be subscribed.

Issue Details

Backport of #51300 to release/6.0-preview4

/cc @layomia

Customer Impact

Testing

Risk

Author: github-actions[bot]
Assignees: -
Labels:

area-System.Text.Json, new-api-needs-documentation

Milestone: -

@layomia layomia self-assigned this Apr 19, 2021
@layomia layomia requested a review from ericstj April 19, 2021 19:01
@layomia layomia assigned Anipik and unassigned Anipik Apr 19, 2021
@layomia layomia requested a review from Anipik April 19, 2021 19:02
@Anipik Anipik enabled auto-merge (squash) April 20, 2021 23:06
@Anipik Anipik disabled auto-merge April 20, 2021 23:07
@Anipik Anipik merged commit 1ce1be7 into release/6.0-preview4 Apr 20, 2021
@akoeplinger akoeplinger deleted the backport/pr-51300-to-release/6.0-preview4 branch April 22, 2021 18:15
@ghost ghost locked as resolved and limited conversation to collaborators May 22, 2021
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.

2 participants