[IO] Build fails with 3 checkstyle errors

classic Classic list List threaded Threaded
4 messages Options
Reply | Threaded
Open this post in threaded view
|

[IO] Build fails with 3 checkstyle errors

Rob Spoor
See https://travis-ci.org/apache/commons-io/jobs/569951808. Two of the
errors report an unused import which is in fact used in Javadoc. The
other is a protected method without Javadoc.

---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

Re: [IO] Build fails with 3 checkstyle errors

garydgregory
Checkstyle is not perfect :-( I will hack that in a little bit...

Gary

On Fri, Aug 9, 2019, 15:22 Rob Spoor <[hidden email]> wrote:

> See https://travis-ci.org/apache/commons-io/jobs/569951808. Two of the
> errors report an unused import which is in fact used in Javadoc. The
> other is a protected method without Javadoc.
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: [hidden email]
> For additional commands, e-mail: [hidden email]
>
>
Reply | Threaded
Open this post in threaded view
|

Re: [IO] Build fails with 3 checkstyle errors

sebb-2-2
I think checkstyle is correct here.

It's confusing to see an import that is only used by Javadoc.

On Fri, 9 Aug 2019 at 21:50, Gary Gregory <[hidden email]> wrote:

>
> Checkstyle is not perfect :-( I will hack that in a little bit...
>
> Gary
>
> On Fri, Aug 9, 2019, 15:22 Rob Spoor <[hidden email]> wrote:
>
> > See https://travis-ci.org/apache/commons-io/jobs/569951808. Two of the
> > errors report an unused import which is in fact used in Javadoc. The
> > other is a protected method without Javadoc.
> >
> > ---------------------------------------------------------------------
> > To unsubscribe, e-mail: [hidden email]
> > For additional commands, e-mail: [hidden email]
> >
> >

---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

Re: [IO] Build fails with 3 checkstyle errors

garydgregory
In reply to this post by garydgregory
All better now.

Gary

On Fri, Aug 9, 2019, 16:50 Gary Gregory <[hidden email]> wrote:

> Checkstyle is not perfect :-( I will hack that in a little bit...
>
> Gary
>
> On Fri, Aug 9, 2019, 15:22 Rob Spoor <[hidden email]> wrote:
>
>> See https://travis-ci.org/apache/commons-io/jobs/569951808. Two of the
>> errors report an unused import which is in fact used in Javadoc. The
>> other is a protected method without Javadoc.
>>
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: [hidden email]
>> For additional commands, e-mail: [hidden email]
>>
>>