ctubbsii commented on code in PR #2583: URL: https://github.com/apache/thrift/pull/2583#discussion_r866294649
########## lib/java/.gitignore: ########## @@ -0,0 +1,53 @@ +# Created by https://www.toptal.com/developers/gitignore/api/java,gradle +# Edit at https://www.toptal.com/developers/gitignore?templates=java,gradle + Review Comment: This change adds a new .gitignore file in the lib/java directory. This is consistent with the recently added one for lib/kotlin. However, they do differ in some ways. And, a lot of this is redundant with the top-level .gitignore file. I think it'd be better to remove both of these .gitignore files and add the relevant patterns into the .gitignore file at the root of the repository, rather than add this here. -- 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]
