gnodet commented on PR #22292:
URL: https://github.com/apache/camel/pull/22292#issuecomment-4141251118
Good point. I've verified that the website docs are still generated with all
correct content:
**Glob pattern equivalence:** Using `npx glob@11`, confirmed both old
(`../dsl/**/src/main/docs/*.adoc` with `ignore: ['**/target/**']`) and new
(`../dsl/src/main/docs/*.adoc` + `../dsl/{*,*/!(target)}/src/main/docs/*.adoc`)
patterns match exactly the same 11 files — no files gained, none lost.
**Full gulp build:** Ran `gulp` end-to-end — completed successfully with all
71 "others" pages generated and all DSL entries present in the generated
`nav.adoc` (dsl, cli-connector, cli-debug, dsl-modeline, java-joor-dsl,
kamelet-main, kamelet-main-support, java-xml-io-dsl, java-xml-jaxb-dsl,
yaml-dsl, camel-yaml-dsl-validator-maven-plugin).
**CI:** Green on this PR.
Regarding build links: this race condition is primarily observable during
local development with mvnd (where docs and dsl modules build in parallel). In
CI, `regen.sh` uses `./mvnw` sequentially, making the race much less likely. No
CI builds were found with this specific ENOENT error. This is a preventive
improvement, consistent with PR #21996 which was also described as a
"preventive change, not a fix for a specific observed failure."
I've updated the PR description with the full validation details.
_Claude Code on behalf of Guillaume Nodet_
--
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]
