asafm opened a new pull request, #23981:
URL: https://github.com/apache/flink/pull/23981
## What is the purpose of the change
Small fix to README
## Brief change log
No need for pre-installed maven, since we have Maven Wrapper
## Verifying this change
Please
flinkbot commented on PR #23981:
URL: https://github.com/apache/flink/pull/23981#issuecomment-1866287852
## CI report:
* 8bac98b51ccb4a13d5a498e377f17bb86b953b1d UNKNOWN
Bot commands
The @flinkbot bot supports the following commands:
- `@flinkbot run azure`
snuyanzin commented on PR #23981:
URL: https://github.com/apache/flink/pull/23981#issuecomment-1866302758
having maven wrapper does not protect from running with installed maven
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub
asafm commented on PR #23981:
URL: https://github.com/apache/flink/pull/23981#issuecomment-1868536450
@snuyanzin Why would you want users, to run with their installed Maven? If
you want all users to have reproducible builds, they should only use the maven
wrapper, pinned to a specific versi
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 wrapp
asafm commented on PR #23981:
URL: https://github.com/apache/flink/pull/23981#issuecomment-1871374396
Guys, I just thought it make sense that if you use internally maven wrapper,
then the getting started README should mention that and not "force" users to
download maven and run the build in
asafm closed pull request #23981: [hotfix] [docs] Update README.md to remove
Maven
URL: https://github.com/apache/flink/pull/23981
--
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 comme
