adoroszlai commented on a change in pull request #11: Update maven versioning URL: https://github.com/apache/ambari-logsearch/pull/11#discussion_r227771380
########## File path: README.md ########## @@ -8,7 +8,7 @@ Log Search is a sub-project of [Apache Ambari](https://github.com/apache/ambari) ## Development -Requires JDK 8 (JDK 11 is recommended) +Requires JDK 8 (JDK 11 is recommended) and Maven 3.5.x Review comment: I think this should be something like "Maven 3.5 (or newer)" to avoid having to update later. When Maven 3.6 is released and it works fine, developers shouldn't be wondering if 3.6 is OK, or do they have to use 3.5 for compatibility reasons (like we used to have some such instructions for Ambari). ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services