[GitHub] [commons-release-plugin] kinow commented on a change in pull request #44: Replace construction of FileInputStream and FileOutputStream objects with Files NIO APIs.

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

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:
users@infra.apache.org