elharo commented on code in PR #113: URL: https://github.com/apache/maven-changes-plugin/pull/113#discussion_r1978032299
########## src/main/java/org/apache/maven/plugins/changes/ChangesReport.java: ########## @@ -75,6 +75,21 @@ public class ChangesReport extends AbstractChangesReport { @Parameter(property = "changes.addActionDate", defaultValue = "false") private boolean addActionDate; + /** + * Whether the change description should be passed as raw text to the Doxia sink or not. Review Comment: Whether the change description should be escaped in the generated report. Set to false if the change descriptions already contain escape sequences such as < or \" You shouldn't mention Doxia here. That is an implementation detail that shouldn't appear in user docs. ########## src/main/java/org/apache/maven/plugins/changes/ChangesReport.java: ########## @@ -75,6 +75,14 @@ public class ChangesReport extends AbstractChangesReport { @Parameter(property = "changes.addActionDate", defaultValue = "false") private boolean addActionDate; + /** + * Whether the changelog action text should be passed as raw text to the Doxia sink or not. + * + * @since 3.0 + */ + @Parameter + private boolean passRawText; Review Comment: I prefer escapeText. I'm not sure if anyone else feels strongly about this. -- 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