Claudenw commented on PR #167: URL: https://github.com/apache/creadur-rat/pull/167#issuecomment-1826258970
First, thank you for your contribution. All in all this looks like a good solution given the current Maven implementation. However, we have a change to the ReportConfiguration to specify a directory pattern to skip (RAT-322). This solution may indicate that that design is incomplete. We are headed down the path of using FileFilters to filter inclusion/exclusion of files and directories. There is also an open ticket to harmonize the UIs (RAT-323) by making sure that all functionality is supported by the `ReportConfiguraiton` and then ensuring that each UI implementation (CLI, Ant, Maven) support all the options. Would you be interested in participating in the discussions around how to denote the inclusion/exclusion of files and directories? -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
