Claudenw commented on code in PR #246: URL: https://github.com/apache/creadur-rat/pull/246#discussion_r1597224503
########## src/changes/changes.xml: ########## @@ -107,12 +119,15 @@ https://maven.apache.org/plugins/maven-changes-plugin/xsd/changes-1.0.0.xsd </action> <action issue="RAT-366" type="update" dev="claudenw"> Switch to processing header matches in one call rather than line by line. + + This change also resulted in the possibility of multiple licenses being detected and reported. forcing a change in the + XML ouptut. XML schema was developed for the output. </action> <action issue="RAT-333" type="fix" dev="claudenw"> Fix if --force option is used executable bit is not set properly on newly created/license-augmented file. </action> <action issue="RAT-345" type="update" dev="pottlinger" due-to="Niels Basjes"> - Update gitignore-reader from 1.4.0 to 1.5.1 to fetch changes resulting from fixes of RAT-362. + Update gitignore-reader from 1.4.0 to 1.5.1 to fetch changes resulting from fixes of RAT-362. Review Comment: The entries had some weird [NBSP] (non break space?) character, I replaced them with a space. I don't think it matters, but it made my IntelliJ editor much happier. -- 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: dev-unsubscr...@creadur.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
