[ 
https://issues.apache.org/jira/browse/DOXIATOOLS-88?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17896882#comment-17896882
 ] 

ASF GitHub Bot commented on DOXIATOOLS-88:
------------------------------------------

kwin commented on code in PR #85:
URL: 
https://github.com/apache/maven-doxia-converter/pull/85#discussion_r1835347143


##########
src/main/java/org/apache/maven/doxia/DefaultConverter.java:
##########
@@ -426,6 +434,10 @@ private void parse(
                 throw new ConverterException("IOException: " + e.getMessage(), 
e);
             }
         }
+        LOGGER.info(

Review Comment:
   For me this is crucial information when a directory has been given as input. 
For a single file it should not matter (as it is only one log line) and also 
clarifies which absolute file has been used (as input parameter is often 
relative).





> Optionally remove input files after conversion
> ----------------------------------------------
>
>                 Key: DOXIATOOLS-88
>                 URL: https://issues.apache.org/jira/browse/DOXIATOOLS-88
>             Project: Maven Doxia Tools
>          Issue Type: Improvement
>          Components: Doxia Converter
>            Reporter: Konrad Windszus
>            Assignee: Konrad Windszus
>            Priority: Major
>             Fix For: doxia-converter-1.4
>
>
> Currently if a directory is given to the {{in}} parameter it is only read. 
> However usually after conversion you want to remove the original input files. 
> There should be a dedicated flag which allows to achieve that automatically.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to