atiaomar1978-hub commented on code in PR #25409:
URL: https://github.com/apache/camel/pull/25409#discussion_r3742023070
##########
archetypes/camel-archetype-api-component/src/main/resources-filtered/META-INF/maven/archetype-metadata.xml:
##########
@@ -63,11 +63,19 @@
</includes>
</fileSet>
+ <!-- project root docs -->
Review Comment:
**Test coverage:** This fileSet fix is exactly the kind of regression
automated archetype ITs would catch — before this PR, ReadMe.txt was listed
under __artifactId__-component/ while the file lived at project root, so no
readme was generated (reproduced on 4.21.0). Consider adding/re-enabling a
build-it archetype IT that asserts README.md and AGENTS.md exist at the
generated project root.
##########
archetypes/camel-archetype-java/src/main/resources/archetype-resources/AGENTS.md:
##########
@@ -0,0 +1,27 @@
+# AGENTS.md
Review Comment:
**Test coverage:** No automated test validates generated AGENTS.md presence
or content. JBang ExportTest already asserts AGENTS.md exists after export
(#24063). Archetypes have IT scaffolding under
src/test/resources/projects-disabled/ but it is disabled — a minimal smoke IT
per archetype would lock this in.
--
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]
