SingingBush commented on code in PR #68: URL: https://github.com/apache/xerces-j/pull/68#discussion_r2561511950
########## src/org/apache/xerces/xs/datatypes/package.html: ########## @@ -27,6 +27,7 @@ <BR> <BR> <TABLE border="1"> + <caption>Mappings between XML Schema Data Type and applicable interfaces</caption> Review Comment: the lack of a caption on a table is not a warning, it's an error. It's probably due to accessibility and although I agree that the linter is over the top in this regard, this is coming from the Javadoc linting within the JDK. It's doing this for accessibility purposes. I cannot comment on how useful these WAI (web accessibility initiative) guidelines are and perhaps the Javadoc task is being over-zealous in it's demands but that is the way it is. I'd previously used empty captions so that the build resulted in warnings instead of errors but that was rejected so I've added appropriate text so that the generated javadocs meets the expected standards. If the concern is about duplicating text then I suggest keeping the caption and removing the plain text above. It would be better to fall in line with standards and making use of the linter helps in that goal -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
