pjfanning commented on code in PR #1489: URL: https://github.com/apache/hamilton/pull/1489#discussion_r2971297932
########## dev_tools/language_server/tests/ls_setup.py: ########## @@ -1,19 +1,20 @@ -############################################################################ -# Copyright(c) Open Law Library. All rights reserved. # -# See ThirdPartyNotices.txt in the project root for additional notices. # Review Comment: Now I see this is another file. Not the conftest.py file. This does look like it has 3rd party source. The source header needs to be retained and our LICENSE needs to mention this file. Can we take all of this license mess into its own issues and PRs and not not try to deal with everything in one PR? It is really messy to have all this happening in one PR. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
