[
https://issues.apache.org/jira/browse/DOXIATOOLS-90?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17897447#comment-17897447
]
ASF GitHub Bot commented on DOXIATOOLS-90:
------------------------------------------
kwin commented on code in PR #87:
URL:
https://github.com/apache/maven-doxia-converter/pull/87#discussion_r1837827121
##########
src/main/java/org/apache/maven/doxia/DefaultConverter.java:
##########
@@ -250,20 +258,86 @@ public void convert(InputFileWrapper input,
OutputFileWrapper output)
File relativeOutputDirectory = new File(
PathTool.getRelativeFilePath(input.getFile().getAbsolutePath(), f.getParent()));
convert(f, input.getEncoding(), input.getFormat(), output,
relativeOutputDirectory);
- cleanUp(input, f);
}
}
+ try {
+ postProcessAllFiles(output.getFormat());
+ } catch (InterruptedException e) {
+ Thread.currentThread().interrupt();
+ throw new ConverterException("Error post processing all files:
" + e.getMessage(), e);
+ } catch (IOException e) {
+ throw new ConverterException("Error post processing all files:
" + e.getMessage(), e);
+ }
Review Comment:
Because stacktrace (and cause) is not emitted by default. This pattern is
used also e.g. in
https://github.com/apache/maven-doxia-converter/pull/87/files/82219f7819769b5e75ef6e0f40e6a23aa875b247#diff-602e7fdec6cff8b7c04882eccbf6704b6be1978a75cf792196d3efde9c9feb3eR249.
If we want to change that lets track that individually.
> Optionally keep Git history on converted files
> ----------------------------------------------
>
> Key: DOXIATOOLS-90
> URL: https://issues.apache.org/jira/browse/DOXIATOOLS-90
> Project: Maven Doxia Tools
> Issue Type: New Feature
> Components: Doxia Converter
> Reporter: Konrad Windszus
> Assignee: Konrad Windszus
> Priority: Major
>
> Using [git mv|https://git-scm.com/docs/git-mv] allows to keep the file's
> history (although using the blame feature won't work as the line numbers are
> off after conversion).
> This is reasonable to use when the input files are managed with Git and the
> converted ones should still be.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
