kinow commented on a change in pull request #44:
URL:
https://github.com/apache/commons-release-plugin/pull/44#discussion_r593801692
##########
File path:
src/main/java/org/apache/commons/release/plugin/mojos/CommonsSiteCompressionMojo.java
##########
@@ -147,8 +148,9 @@ private void getAllSiteFiles(final File siteDirectory,
final List<File> filesToC
*/
private void writeZipFile(final File outputDirectory, final File
directoryToZip, final List<File> fileList)
throws IOException {
- try (FileOutputStream fos = new
FileOutputStream(outputDirectory.getAbsolutePath() + "/site.zip");
- ZipOutputStream zos = new ZipOutputStream(fos)) {
+ try (OutputStream fos = Files.newOutputStream(new
File(outputDirectory.getAbsolutePath() + "/site.zip")
+ .toPath());
+ ZipOutputStream zos = new ZipOutputStream(fos)) {
Review comment:
Nit-picking, but I think there's one extra space before
`ZipOutputStream`, or at least looking at the GitHub interface it appears to
have an extra 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:
[email protected]
