Skip to content

Conversation

@eNeRGy164
Copy link
Contributor

Fill out details for the System.Text.Encodings.Web NuGet package readme.

See #92228

@ghost ghost added community-contribution Indicates that the PR has been added by a community member area-System.Text.Encodings.Web labels Nov 11, 2023
@ghost
Copy link

ghost commented Nov 11, 2023

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

Issue Details

Fill out details for the System.Text.Encodings.Web NuGet package readme.

See #92228

Author: eNeRGy164
Assignees: -
Labels:

area-System.Text.Encodings.Web, community-contribution

Milestone: -

Copy link
Member

@tarekgh tarekgh 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. Thanks @eNeRGy164 for your help with this issue.

@tarekgh tarekgh merged commit d711400 into dotnet:PackageReadmesContinued Nov 11, 2023
@eNeRGy164 eNeRGy164 deleted the system-text-encodings-web branch November 11, 2023 20:55
using System.Text.Encodings.Web;
using System.Text.Unicode;

TextEncoderSettings customEncoderSettings = new TextEncoderSettings();
Copy link
Member

Choose a reason for hiding this comment

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

nit: it would be good to use the simplified new operator on the right hand side: new();

@github-actions github-actions bot locked and limited conversation to collaborators Dec 12, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-System.Text.Encodings.Web community-contribution Indicates that the PR has been added by a community member

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants