While #4779 tracks trailing white-space clean-up. This issue tracks removing BOM and xml meta-tag from all the file in the Repo.
Update EditorConfig
First, we should update EditorConfig to add charset and set it to utf-8 to prevent future files marked with BOM (Byte Order Mark).
We should also segregate files by category to enable charset or spacing changes one set at a time thereby reducing risk.
Remove BOM from all files in the Repo
As removing BOM from all the files at once is such a troublesome task; It could fix many lingering issues related to BOM files in MSBuild; It could also introduce regressions. Since, MSBuild have fixed several issues related to BOM; It may be not complete, with precaution, we'll remove BOM case by case basis knowing each category of BOM removed files won't cause trouble down the line!
Remove xmlns and meta-tags from all XML files in the Repo
As of MSBuild v15, xml/xaml build files doesn't need to specify the default xmlns though there were issues with temporary project generation (in WPF/Workflow) requiring xmlns. Since, those issues have been fixed, it's safe to remove them.
For xml meta-tag, the default has been utf-8 and v1 for so... long, I don't remember when. Thus, we don't need those meta-tag cluttering the top of the files.
<?xml version="1.0" encoding="utf-8"?>
Notes
If there's anything I missed please let me know.
While #4779 tracks trailing white-space clean-up. This issue tracks removing BOM and xml meta-tag from all the file in the Repo.
Update EditorConfig
First, we should update EditorConfig to add
charsetand set it toutf-8to prevent future files marked with BOM (Byte Order Mark).We should also segregate files by category to enable charset or spacing changes one set at a time thereby reducing risk.
charsetto EditorConfigRemove BOM from all files in the Repo
As removing BOM from all the files at once is such a troublesome task; It could fix many lingering issues related to BOM files in MSBuild; It could also introduce regressions. Since, MSBuild have fixed several issues related to BOM; It may be not complete, with precaution, we'll remove BOM case by case basis knowing each category of BOM removed files won't cause trouble down the line!
Remove xmlns and meta-tags from all XML files in the Repo
As of MSBuild v15, xml/xaml build files doesn't need to specify the default xmlns though there were issues with temporary project generation (in WPF/Workflow) requiring xmlns. Since, those issues have been fixed, it's safe to remove them.
For xml meta-tag, the default has been utf-8 and v1 for so... long, I don't remember when. Thus, we don't need those meta-tag cluttering the top of the files.
Notes
If there's anything I missed please let me know.