Re: Bumping Checkstyle plugin version to 3.1.0-SNAPSHOT (from 3.0.1-SNAPSHOT)

2019-04-23 Thread Tibor Digana
+1 a good job, I agree with 3.1.0.

On Sun, Apr 21, 2019 at 5:24 PM Enrico Olivelli  wrote:

> Hi,
> I want to commit this breaking change [1] in checkstyle plugin,
> that is the upgrade [2] to latest checkstyle (8.x) and thus the making
> Java 8 the minimum version for the plugin.
>
> Currently I have 3.0.1-SNAPSHOT on master,
> I think I need to move to 3.1.0-SNAPSHOT.
>
> Is is better to bump the version to 3.1.0-SNAPSHOT and then commit the
> breaking change or commit the change and then commit the new version ?
>
> I feel it is better to change the version to 3.1.0-SNAPSHOT and then
> commit the breaking change.
>
> Any review on the PR is well appreciated as well
>
> Best regards and Happy Easter !
> Enrico
>
> [1] https://github.com/apache/maven-checkstyle-plugin/pull/13
> [2] https://issues.apache.org/jira/browse/MCHECKSTYLE-366
>
> -
> To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
> For additional commands, e-mail: dev-h...@maven.apache.org
>
>


Re: Bumping Checkstyle plugin version to 3.1.0-SNAPSHOT (from 3.0.1-SNAPSHOT)

2019-04-23 Thread Karl Heinz Marbaise

Hi Enrico,

Done so.

Kind regards
Karl Heinz Marbaise

On 23.04.19 00:26, Enrico Olivelli wrote:

Committed.
Can any JIRA admin rename version 3.0.1 to 3.1.0 please ?

Thanks
Enrico

Il dom 21 apr 2019, 21:07 Stephen Connolly 
ha scritto:


On Sun 21 Apr 2019 at 16:24, Enrico Olivelli  wrote:


Hi,
I want to commit this breaking change [1] in checkstyle plugin,
that is the upgrade [2] to latest checkstyle (8.x) and thus the making
Java 8 the minimum version for the plugin.

Currently I have 3.0.1-SNAPSHOT on master,
I think I need to move to 3.1.0-SNAPSHOT.



+1




Is is better to bump the version to 3.1.0-SNAPSHOT and then commit the
breaking change or commit the change and then commit the new version ?

I feel it is better to change the version to 3.1.0-SNAPSHOT and then
commit the breaking change.

Any review on the PR is well appreciated as well

Best regards and Happy Easter !
Enrico

[1] https://github.com/apache/maven-checkstyle-plugin/pull/13
[2] https://issues.apache.org/jira/browse/MCHECKSTYLE-366

-
To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
For additional commands, e-mail: dev-h...@maven.apache.org

--


-
To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
For additional commands, e-mail: dev-h...@maven.apache.org



Re: Bumping Checkstyle plugin version to 3.1.0-SNAPSHOT (from 3.0.1-SNAPSHOT)

2019-04-22 Thread Enrico Olivelli
Committed.
Can any JIRA admin rename version 3.0.1 to 3.1.0 please ?

Thanks
Enrico

Il dom 21 apr 2019, 21:07 Stephen Connolly 
ha scritto:

> On Sun 21 Apr 2019 at 16:24, Enrico Olivelli  wrote:
>
> > Hi,
> > I want to commit this breaking change [1] in checkstyle plugin,
> > that is the upgrade [2] to latest checkstyle (8.x) and thus the making
> > Java 8 the minimum version for the plugin.
> >
> > Currently I have 3.0.1-SNAPSHOT on master,
> > I think I need to move to 3.1.0-SNAPSHOT.
>
>
> +1
>
>
> >
> > Is is better to bump the version to 3.1.0-SNAPSHOT and then commit the
> > breaking change or commit the change and then commit the new version ?
> >
> > I feel it is better to change the version to 3.1.0-SNAPSHOT and then
> > commit the breaking change.
> >
> > Any review on the PR is well appreciated as well
> >
> > Best regards and Happy Easter !
> > Enrico
> >
> > [1] https://github.com/apache/maven-checkstyle-plugin/pull/13
> > [2] https://issues.apache.org/jira/browse/MCHECKSTYLE-366
> >
> > -
> > To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
> > For additional commands, e-mail: dev-h...@maven.apache.org
> >
> > --
> Sent from my phone
>


Re: Bumping Checkstyle plugin version to 3.1.0-SNAPSHOT (from 3.0.1-SNAPSHOT)

2019-04-21 Thread Stephen Connolly
On Sun 21 Apr 2019 at 16:24, Enrico Olivelli  wrote:

> Hi,
> I want to commit this breaking change [1] in checkstyle plugin,
> that is the upgrade [2] to latest checkstyle (8.x) and thus the making
> Java 8 the minimum version for the plugin.
>
> Currently I have 3.0.1-SNAPSHOT on master,
> I think I need to move to 3.1.0-SNAPSHOT.


+1


>
> Is is better to bump the version to 3.1.0-SNAPSHOT and then commit the
> breaking change or commit the change and then commit the new version ?
>
> I feel it is better to change the version to 3.1.0-SNAPSHOT and then
> commit the breaking change.
>
> Any review on the PR is well appreciated as well
>
> Best regards and Happy Easter !
> Enrico
>
> [1] https://github.com/apache/maven-checkstyle-plugin/pull/13
> [2] https://issues.apache.org/jira/browse/MCHECKSTYLE-366
>
> -
> To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
> For additional commands, e-mail: dev-h...@maven.apache.org
>
> --
Sent from my phone