Jamison929611 commented on PR #2060: URL: https://github.com/apache/maven-resolver/pull/2060#issuecomment-5353895864
Fixed in 4f21cd46. The missing link was caused by internalJavadocSourceRoots being derived only from source files that declare configuration keys. That excluded sibling reactor modules whose types are part of the aggregate Javadoc. The collector now: - keeps the doclet compilation units limited to marker-bearing configuration sources; - derives aggregate Javadoc membership from all reactor src/main/java source roots; - detects those roots using Path operations so the collection also works on Windows. I added an end-to-end regression test where a configuration source in one module references ConfigurationProperties.HttpVersion from a sibling module. The targeted reactor package run passes with 9 tests, and the real generated target/generated-site/markdown/configuration.md now contains both the ConfigurationProperties.HttpVersion Javadoc link and the #aether.transport.http.version configuration anchor. @kwin, could you please take another look? -- 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]
