Let's start out with external javadocs only. That way the focus is on the
users of Geode. Sound good?

-Kirk


On Mon, Aug 15, 2016 at 3:57 PM, Mark Bretl <asf.mbr...@gmail.com> wrote:

> I can add 'geode-assembly/build/install/' to the archive step in the
> nightly build or we could create a separate job, if necessary, to add the
> external javadocs. If we add the external javadocs to the current nightly
> build, the perma link would be
> https://builds.apache.org/job/Geode-nightly/lastSuccessfulBuild/artifact/
> geode-assembly/build/install/apache-geode/javadoc/index.html.
> It is only 19MB in size for the external docs.
>
> If we want to add internal ( **/build/docs/javadoc ), that goes up to
> 211MB.
>
> --Mark
>
> On Sun, Aug 14, 2016 at 8:21 PM, Udo Kohlmeyer <ukohlme...@pivotal.io>
> wrote:
>
> > Maybe the review of the Javadoc for any class should be done as part of
> > the final review of a story.
> >
> > Any public facing classes should be checked that the javadoc is correct
> > for the affected classes.
> >
> > That the render correctly in the IDE, I think might be a side affect of
> > making sure that they docs are actually checked.
> >
> > --Udo
> >
> >
> >
> > On 13/08/2016 10:00 AM, Kirk Lund wrote:
> >
> >> We really need to find an automated solution for discovering javadoc
> >> errors
> >> earlier than manual inspection or looking for warnings in the gradle
> >> output.
> >>
> >> We change our build to fail on javadoc errors because it starts to fail
> >> due
> >> to 3rd party library javadocs, but at a minimum there must be some
> >> alternative for enforcing no warnings in the build.
> >>
> >> As for ConfigurationProperties' javadocs, I don't think it's generating
> >> any
> >> warnings but the javadocs contain malformed html so the result is
> clearly
> >> broken. I'm not sure if there's anyway to automate detecting problems
> like
> >> that.
> >>
> >> -Kirk
> >>
> >>
> >> On Fri, Aug 12, 2016 at 4:51 PM, Dan Smith <dsm...@pivotal.io> wrote:
> >>
> >> I'm not sure if we're publishing the nightly javadocs anywhere, but you
> >>> can
> >>> build them from source. If you do a ./gradlew build, the external
> >>> javadocs
> >>> are in geode-assembly/build/install/apache-geode/javadoc
> >>>
> >>> -Dan
> >>>
> >>> On Fri, Aug 12, 2016 at 4:31 PM, Kirk Lund <kl...@pivotal.io> wrote:
> >>>
> >>> I can find the following from Google, but this is the javadocs for the
> M2
> >>>> release:
> >>>>
> >>>> http://geode.incubator.apache.org/releases/latest/javadoc/
> >>>>
> >>>> Does this need manual updating for M3?
> >>>>
> >>>> Are we publishing nightly javadocs?
> >>>>
> >>>> When I use CTRL-J (in IntelliJ) on the ConfiguratingProperties class,
> >>>> the
> >>>> javadocs look malformed, so I'd like to review real javadoc html for
> the
> >>>> current state of develop.
> >>>>
> >>>> -Kirk
> >>>>
> >>>>
> >
>

Reply via email to