vy commented on PR #1441:
URL: https://github.com/apache/logging-log4j2/pull/1441#issuecomment-2275059321
> We could add an option to `log4j-docgen-maven-plugin` to use `xsd:string`
for all attribute, but we wouldn't use the option in the XSD schemas available
on our site.
I would not be in favor of this change. This will practically remove the IDE
guidance provided using the XSD, which IMHO, is the point of publishing an XSD
in our case. OTOH, I see the point that `level="${foo}"` is syntactically
invalid according to the XSD. If IDE could have recognized `log4j2.xml` syntax
(just like it does for Spring Boot external configuration files), I doubt if
@krallus will still be complaining. :thinking:
@krallus, are you trying to (manually) validate your `log4j2.xml` against
the XSD, or you're only concerned about IDE complaints?
--
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]
