vy commented on issue #1954: URL: https://github.com/apache/logging-log4j2/issues/1954#issuecomment-1929270262
Great work Piotr! :100: :clap: Some remarks: 1. I see many pages are hardly filled. Shall we collapse the entire reference to a single page? WDYT? That is, one page per [Maven] module? 1. Why do some nested components appear as `<AppenderRef/>` while some `... one Filter element ...`? I was expect the latter to be formatted as `<Filter/>` too. 1. Looking at [AsyncAppender](https://logging.staged.apache.org/log4j/3.x/ref/Core/org.apache.logging.log4j.core.appender.AsyncAppender.html), I cannot see * if I can provide one or more `AppenderRef`s * what is the default for a nested component 1. We need to document the `KeyValuePair` component 1. We need to document the `value` field of the `Property` component > Also the links in the _descriptions_ (which are taken from `{@link ...}` comments in Javadoc) are usually broken, since the documentation generator currently can not determine the correct link to use. What is the plan to address this shortcoming? -- 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]
