snuyanzin commented on PR #23981: URL: https://github.com/apache/flink/pull/23981#issuecomment-1868594128
1. the PR is incomplete: removal requirement for maven is not enough since to make it reproducible there is no info how to enable it in IDE. In that case in terminal there will be wrapper used and in IDE not if follow instructions only 2. across the documentation there are lots of of mentions of just `mvn ...` without mentioing `./mvnw` 3. Is there any issue which could be reproduced by wraper and not reproduced by installed maven or vice versa or what is the reason to have the restriction to wrapper only? 4. Should we have a restriction for maven-wrapper version then? -- 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: issues-unsubscr...@flink.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
