Claudenw commented on code in PR #233: URL: https://github.com/apache/creadur-rat/pull/233#discussion_r1564527324
########## apache-rat-core/src/main/resources/org/apache/rat/plain-rat.xsl: ########## @@ -53,42 +53,58 @@ Files with unapproved licenses: </xsl:if> <xsl:if test="descendant::resource[type/@name='archive']"> Archives: -<xsl:for-each select='descendant::resource[type/@name="archive"]'> +<xsl:for-each select='descendant::resource[@type="archive"]'> + <xsl:value-of select='@name'/> <xsl:text> </xsl:text> </xsl:for-each> </xsl:if> +<xsl:text> ***************************************************** - Files with Apache License headers will be marked AL - Binary files (which do not require any license headers) will be marked B - Compressed archives will be marked A - Notices, licenses etc. will be marked N + Documents with unapproved licenses will start with a '!' + The next character identifies the document type. + + char type + a Archive file Review Comment: Uppercase is now the default since the enums were changed. -- 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