XenoAmess edited a comment on pull request #137: URL: https://github.com/apache/commons-io/pull/137#issuecomment-683300547
@garydgregory > When I run `mvn site -P jacoco` on git. master, this method already has 100% code coverage, so more tests are not needed IMO. Hi. The new added tests are for pr https://github.com/apache/commons-io/pull/118 Sebb think I should not change test codes in that pr (in order not to break any test, which I agree somehow), so I have to split them out to a new pr. So this pr SHOULD be reviewed only after https://github.com/apache/commons-io/pull/118 ---------------------------------------------------------------- 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: [email protected]
