DonalEvans commented on code in PR #7594:
URL: https://github.com/apache/geode/pull/7594#discussion_r851535875


##########
geode-assembly/build.gradle:
##########
@@ -384,7 +384,7 @@ tasks.register('gfshDepsJar', Jar) {
 
 tasks.register('docs', Javadoc) {
   def docsDir = file("$buildDir/javadocs")
-  options.addStringOption('Xdoclint:none', '-quiet')
+  options.addStringOption('Xwerror', '-quiet')

Review Comment:
   No, the build currently passes, and will only fail if someone breaks 
existing javadocs or introduces new javadocs that have errors or warnings. 
Since we shouldn't be introducing javadoc warnings or errors, making it a 
requirement that they be fixed seems like a good idea.



-- 
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: notifications-unsubscr...@geode.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to