Yes, my main concern is: why does the maven succeed, if we're still having javadoc issues?
Andor -----Original Message----- From: Enrico Olivelli <eolive...@gmail.com> Reply-To: dev@zookeeper.apache.org To: dev@zookeeper.apache.org Subject: Re: Intellij Idea warning as error with -Xdoclint Date: Mon, 15 Jul 2019 21:58:09 +0200 Il lun 15 lug 2019, 13:53 Andor Molnar < an...@apache.org > ha scritto: > Hi Enrico, > > This is still an issue for me in IDEA: > > Javadoc problems found in zookeeper-jute and zookeeper-server > projects, so > the build eventually fails. Strange thing is that running maven from > command line doesn’t show any warning nor error. > > From javac params -Werror and -Xdoclint I think IDEA’s behaviour is > correct. > > Most warning messages in IDEA are about missing javadoc comments, so > changing it to "-Xdoclint:-missing” solves the problem. > I can try to understand better. If I understand correctly your point is that apparently Idea is working as expected but Maven not and this is not good because CI runs with Maven. I will be back with news. Hopefully we can fix javadocs and have a reliable way to detect issues on docs on CI/precommit/nightly Enrico > Regards, > Andor > > > > > > On 2019. Jun 20., at 20:28, Andor Molnar < > > an...@apache.org > > > wrote: > > > > Done. #983 merged, problem gone. > > Thanks maoling! > > > > Andor > > > > > > > > > On 2019. Jun 20., at 15:43, maoling < > > > maoling199210...@sina.com > > > > wrote: > > > > > > I also noticed this when running UT with IDEA.may we can merge > > > this PR: > > > https://github.com/apache/zookeeper/pull/988 > > > > > > Or this one: > > > https://github.com/apache/zookeeper/pull/983 > > > to remove > > that warning. > > > 发送自 Windows 10 版邮件应用 > > > > > > 发件人: Andor Molnar > > > 发送时间: 2019年6月20日 5:46 > > > 收件人: DevZooKeeper > > > 主题: Intellij Idea warning as error with -Xdoclint > > > > > > > > > Hi Enrico, > > > > > > I have the following error message in Idea since -Xdoclint is > > > enabled > > in the main pom.xml file: > > > > > /Users/andormolnar/git/my-zookeeper/zookeeper- > jute/src/main/java/org/apache/jute/Utils.java > > > Error:(194, 15) java: @param name not found > > > Error:(231, 15) java: @param name not found > > > > > > Strange that I don’t see the same warnings in console when > > > running ‘mvn > > install’. > > > I confirm that removing “-Xdoclint” (or fixing javadoc issues) > > > solves > > the problem. > > > Andor > > > > > > > > > > > > > >
