Tibor17 edited a comment on issue #217: Set properties readonly where it doesn't make sense to change values URL: https://github.com/apache/maven-surefire/pull/217#issuecomment-457964135 @britter @eolivelli Additionally, we should include these as well: https://github.com/britter/maven-surefire/blob/185e36672a875f15ab121bf54a06a8dc720eae34/maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/AbstractSurefireMojo.java#L207 https://github.com/britter/maven-surefire/blob/185e36672a875f15ab121bf54a06a8dc720eae34/maven-surefire-plugin/src/main/java/org/apache/maven/plugin/surefire/SurefirePlugin.java#L53 https://github.com/britter/maven-surefire/blob/185e36672a875f15ab121bf54a06a8dc720eae34/maven-failsafe-plugin/src/main/java/org/apache/maven/plugin/failsafe/VerifyMojo.java#L113 We have to check in the history why this field exists: https://github.com/britter/maven-surefire/blob/185e36672a875f15ab121bf54a06a8dc720eae34/maven-failsafe-plugin/src/main/java/org/apache/maven/plugin/failsafe/IntegrationTestMojo.java#L62 I think it should be deleted.
---------------------------------------------------------------- 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: [email protected] With regards, Apache Git Services
