romani commented on code in PR #141:
URL: 
https://github.com/apache/maven-checkstyle-plugin/pull/141#discussion_r1630000160


##########
src/main/java/org/apache/maven/plugins/checkstyle/AbstractCheckstyleReport.java:
##########
@@ -26,22 +26,24 @@
 import java.io.OutputStream;
 import java.nio.file.Path;
 import java.util.ArrayList;
+import java.util.Collections;
 import java.util.List;
 import java.util.Locale;
 import java.util.Map;
 
+import 
com.puppycrawl.tools.checkstyle.AbstractAutomaticBean.OutputStreamOptions;

Review Comment:
   Deprecation happened on 
https://checkstyle.org/releasenotes.html#Release_10.9.3
   https://github.com/checkstyle/checkstyle/issues/5337
   
   AutomaticBean class is not removed yet , but will be at 
https://github.com/checkstyle/checkstyle/issues/12873
   
   Usage in this file:
   ```
   if ("xml".equals(outputFileFormat)) {
                   listener = new XMLLogger(out, OutputStreamOptions.CLOSE);
               } else if ("plain".equals(outputFileFormat)) {
                   listener = new DefaultLogger(out, OutputStreamOptions.CLOSE);
               } else if ("sarif".equals(outputFileFormat)) {
                   try {
                       listener = new SarifLogger(out, 
OutputStreamOptions.CLOSE);
                   } catch (IOException e) {
                       throw new MavenReportException("Failed to create 
SarifLogger", e);
                   }
               } else {
                   // TODO: failure if not a report
                   throw new MavenReportException(
                           "Invalid output file format: (" + outputFileFormat + 
"). Must be 'plain', 'sarif' or 'xml'.");
               }
   ```
   ...to be extended ...



-- 
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

Reply via email to