[ https://issues.apache.org/jira/browse/MCHANGELOG-119?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17907293#comment-17907293 ]
ASF GitHub Bot commented on MCHANGELOG-119: ------------------------------------------- juulhobert commented on code in PR #21: URL: https://github.com/apache/maven-changelog-plugin/pull/21#discussion_r1893586195 ########## src/main/java/org/apache/maven/plugins/changelog/ChangeLogReport.java: ########## @@ -1218,9 +1224,13 @@ private void doChangedSetDetail(ChangeSet entry, Sink sink) { sink.tableCell_(); sink.tableCell(); - // doRevision( entry.getFiles(), bundle, sink ); - doChangedFiles(entry.getFiles(), sink); - sink.lineBreak(); + + if (displayFileAndRevInfo) { + // doRevision( entry.getFiles(), bundle, sink ); Review Comment: Removed the commented code, thanks for spotting it > Add option to disable displaying file and file revision information on report > ----------------------------------------------------------------------------- > > Key: MCHANGELOG-119 > URL: https://issues.apache.org/jira/browse/MCHANGELOG-119 > Project: Maven Changelog Plugin > Issue Type: New Feature > Affects Versions: 2.2 > Reporter: Ivan Mrva > Priority: Major > Attachments: changelog-disable-file-and-rev-info.diff, > sample-report-with-disabled-file-and-rev-info.png > > > Sometimes, it is better to not display all the information on the HTML > report, such as file and file revision information, which makes the report > unnecessarily long and less readable. For example, in our company, we would > like to use this report as a changelog not only for developers, but also for > testers, etc., who really don't care about concrete changed files and their > revisions. > This feature is especially useful, if used in connection with feature > described in MCHANGELOG-118. > I am submitting a very simple patch that implements this feature by providing > one new configuration parameter: > * *displayFileAndRevInfo* - If true, file and file revision information is > displayed for each SCM entry. > ** default value: true - so current behavior of report generation will not be > changed, unless explicitly specified in configuration > Patch also contains integration tests that test if report is created, if this > param is used. > Example report without file and file revision information is displayed on > attached screenshot. -- This message was sent by Atlassian Jira (v8.20.10#820010)