Skip to content

remove utf8 bom#223

Open
salix5 wants to merge 3 commits intoIceYGO:masterfrom
salix5:patch-bom
Open

remove utf8 bom#223
salix5 wants to merge 3 commits intoIceYGO:masterfrom
salix5:patch-bom

Conversation

@salix5
Copy link
Contributor

@salix5 salix5 commented Feb 12, 2026

Using BOM is neither required nor recommended in utf8.
@mercury233

@salix5
Copy link
Contributor Author

salix5 commented Feb 12, 2026

By the way, the newline char in BotWrapper/BotWrapper.csproj are wrong.

@mercury233
Copy link
Collaborator

https://learn.microsoft.com/en-us/visualstudio/msbuild/common-msbuild-project-properties?view=visualstudio

I think we should also add <CodePage>65001</CodePage>.

@salix5
Copy link
Contributor Author

salix5 commented Mar 1, 2026

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants