Copilot commented on code in PR #535: URL: https://github.com/apache/maven-ear-plugin/pull/535#discussion_r3644964822
########## src/site/apt/examples/specifying-env-entries-for-the-generated-application-xml.apt.vm: ########## @@ -1,5 +1,5 @@ ------ - Specifying Security Roles For The Generated application.xml + Specifying Environment entries For The Generated application.xml Review Comment: The top document title uses sentence casing for "entries", but other example titles use title case for key nouns (e.g., "Security Roles", "Resource Refs"). Consider capitalizing "Entries" for consistency across the examples. ########## src/site/apt/examples/specifying-env-entries-for-the-generated-application-xml.apt.vm: ########## @@ -24,7 +24,7 @@ ~~ http://maven.apache.org/doxia/references/apt-format.html -Specifying Environment entries For The Generated application.xml +Specifying Environment entries for the Generated application.xml Review Comment: The section heading capitalization ("for the") is inconsistent with the surrounding examples and with the document title above (which use "For The"). This looks like an accidental casing change rather than part of the title fix. -- 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]
