[GitHub] [fineract] vorburger commented on a change in pull request #587: minor README clean up to remove more left over Gradle wrapper mentions

[GitBox <gi...@apache.org>]

vorburger commented on a change in pull request #587: minor README clean up to remove more left over Gradle wrapper mentions
URL: https://github.com/apache/fineract/pull/587#discussion_r293794263
 
 

 ##########
 File path: README.md
 ##########
 @@ -10,14 +10,12 @@ Requirements
 Instructions to run Apache RAT (Release Audit Tool)
 ============
 1. Extract the archive file to your local directory.
-2. Download gradle-wrapper.jar version 2.10 and place it in the fineract-provider/gradle/wrapper folder. See 'Instructions to download gradle wrapper' above.
-3. Run `./gradlew rat`. A report will be generated under build/reports/rat/rat-report.txt
 
 Review comment:
   Oh, I see... yeah, that was done in #581 (FYI @awasum), sorry about that. I've partially reverted it, and amended the text so that it's clearer - how about #591 ? With that, perhaps we don't need to repeat it N times in the README (and have old 2.10 version numbers which are wrong and won't be maintained..), what do you think? If that makes sense to you, we could still merge this one?

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services