Thanks for the update Bruno :-)
I wish both dependabot and the maven dependency plugin took into account
Java versions...
Gary
On Sat, May 21, 2022, 18:40 Bruno Kinoshita wrote:
> Ah, good point. I saw builds in other projects failing, and the PR's being
> closed due to the upgrade, but
There you go: https://github.com/apache/commons-text/pull/328 take a look
and feel free to merge it if it's looking OK, please
I tested first by making GH Actions to run just `mvn`, without changing the
pom.xml, and the build failed. Then reverted the change in pom.xml and
push-forced, and now
Ah, good point. I saw builds in other projects failing, and the PR's being
closed due to the upgrade, but looks like Text's GH Actions set up needs to
be fixed too. The build for checkstyle passed (even though it appears to
include Java 8), maybe it's not calling checkstyle.
rs List
> Subject: Re: [commons-text] branch master updated: Bump checkstyle from 9.3
> to 10.2
>
> -1
>
> Please stop updating checkstyle to 10.2 because it breaks the build as
> 10.2 requires Java 11.
>
> Gary
>
> On Fri, May 20, 2022 at 7:04 PM wrote:
> >
&g
So at what point do we consider updating to new required Java version?
From: Gary Gregory
Sent: Saturday, May 21, 2022 8:48 AM
To: Commons Developers List
Subject: Re: [commons-text] branch master updated: Bump checkstyle from 9.3 to
10.2
-1
Please stop
I suggest the Pom is commented accordingly
On Sat, 21 May 2022 at 14:48, Gary Gregory wrote:
>
> -1
>
> Please stop updating checkstyle to 10.2 because it breaks the build as
> 10.2 requires Java 11.
>
> Gary
>
> On Fri, May 20, 2022 at 7:04 PM wrote:
> >
> > This is an automated email from the
-1
Please stop updating checkstyle to 10.2 because it breaks the build as
10.2 requires Java 11.
Gary
On Fri, May 20, 2022 at 7:04 PM wrote:
>
> This is an automated email from the ASF dual-hosted git repository.
>
> kinow pushed a commit to branch master
> in repository
