Skip to content

Conversation

rsking
Copy link
Contributor

@rsking rsking commented Jul 22, 2024

This allows an .editorconfig or .globalconfig to unset the charset value.

As it stands at the moment, setting this to unset, or ``, or any invalid value causes the Encoding to be set to UTF8.

@rsking rsking requested a review from a team as a code owner July 22, 2024 03:38
@ghost ghost added Area-AspNetCore RazorSDK, BlazorWebAssemblySDK, dotnet-watch untriaged Request triage from a team member labels Jul 22, 2024
Copy link
Contributor

Thanks for your PR, @rsking.
To learn about the PR process and branching schedule of this repo, please take a look at the SDK PR Guide.

@baronfel baronfel added Area-Format and removed Area-AspNetCore RazorSDK, BlazorWebAssemblySDK, dotnet-watch labels Jul 22, 2024
Copy link
Contributor

Due to lack of recent activity, this PR has been labeled as 'Stale'. It will be closed if no further activity occurs within 7 more days. Any new comment will remove the label.

@github-actions github-actions bot added the stale label Sep 22, 2025
@JoeRobich
Copy link
Member

Thanks @rsking!

@JoeRobich JoeRobich enabled auto-merge September 22, 2025 17:22
@Frulfump
Copy link

Will this miss .NET 10 due to the late review or can it be backported for RC2?

@JoeRobich
Copy link
Member

Will this miss .NET 10 due to the late review or can it be backported for RC2?

Happy to backport. Is a very low risk fix.

@JoeRobich
Copy link
Member

/backport to release/10.0.1xx

Copy link
Contributor

Started backporting to release/10.0.1xx: https://github.com/dotnet/sdk/actions/runs/17923396510

@JoeRobich
Copy link
Member

/azp run dotnet-sdk-public-ci

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@JoeRobich JoeRobich merged commit 6175995 into dotnet:main Sep 25, 2025
27 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area-Format untriaged Request triage from a team member
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants