RyanSkraba commented on a change in pull request #43: Refactored the template configuration to a common parent and introduced Maven archetype URL: https://github.com/apache/incubator-training/pull/43#discussion_r354190699
########## File path: tools/content-parent-pom/pom.xml ########## @@ -75,7 +85,7 @@ <configuration> <consoleOutput>true</consoleOutput> <excludes> - <!-- TODO: This is a mofified version of an asciidoctor file ... nor sure we can stick an Apache Header on it --> + <!-- TODO: This is a modified version of an asciidoctor file ... nor sure we can stick an Apache Header on it --> Review comment: ```suggestion <!-- TODO: This is a modified version of an asciidoctor file ... not sure we can stick an Apache Header on it --> ``` While we're at it.. :D What would it take to be sure? ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services