CurtHagenlocher commented on code in PR #45753: URL: https://github.com/apache/arrow/pull/45753#discussion_r1992165936
########## csharp/src/Apache.Arrow.Flight/FlightInfo.cs: ########## @@ -1,4 +1,4 @@ -// Licensed to the Apache Software Foundation (ASF) under one or more +// Licensed to the Apache Software Foundation (ASF) under one or more Review Comment: It appears to be removing the "UTF-8 BOM". I don't know what the state of all the other source files is in this regard; I know that it's generally not recommended to use one (and I expect it's not required to compile C#) but I know there are definitely things in the Microsoft world that work better when there is one. On the whole, if we think we should get rid of these, we should probably do it for all the files in one change. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: github-unsubscr...@arrow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
