-
Notifications
You must be signed in to change notification settings - Fork 571
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
Enforce UTF-8 no BOM on files #2059
Comments
Is |
yes, we currently enforce reverse :-) we need to convert all files as part of this issue :-) Better if we add some simple check as part of the build (not required though to mark this as complete) |
Do we have a documentation why this should be an improvement? I'm still fearing that removing the BOM will result in different editors/OSes breaking non-ascii characters in our code files. And we have quite a few of those. |
We should remove BOM from all files and add some enforcement so we don't change it back by accident.
The text was updated successfully, but these errors were encountered: