thompson-tomo commented on code in PR #46624:
URL: https://github.com/apache/arrow/pull/46624#discussion_r2113138464
##########
csharp/Directory.Build.props:
##########
@@ -45,7 +45,7 @@
<Authors>The Apache Software Foundation</Authors>
<PackageIcon>feather.png</PackageIcon>
<!-- We can't use PackageLicenseExpression; the license file also contains
3rd-party notices. -->
Review Comment:
Yes but when you review the third party notices, they are referring to the
other languages ie cpp. At the same time on reviewing the source code, I
couldn't find a source file which didn't have the Apache licence header or
wasn't auto generated. Based on that it should be possible to move most if not
all packages from the deprecated property to the replacement.
--
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
