bmarwell commented on code in PR #5: URL: https://github.com/apache/maven-scripting-plugin/pull/5#discussion_r853402558
########## README.md: ########## @@ -65,7 +65,7 @@ There are some guidelines which will make applying PRs easier for us: Optional supplemental description. ``` + Make sure you have added the necessary tests (JUnit/IT) for your changes. -+ Run all the tests with `mvn -Prun-its verify` to assure nothing else was accidentally broken. ++ Run all the tests with `mvn -Prun-its verify` to assure nothing else was accidentally broken (you may need to run `mvn install` beforehand). Review Comment: Review hint: This was not introduced by @rmannibucau’s change, mrm+invoker was already broken before. -- 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...@maven.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org