Re: [ofbiz-framework] branch trunk updated: Improved: Handle remaining checkstyle errors (OFBIZ-12169)

2021-05-25 Thread Jacques Le Roux
Hi, Thanks to nmancus1's help at https://github.com/checkstyle/checkstyle/issues/10012, I finally don't know if it's a checkstyle config issue or a bug in checkstyle. I doubt it's the later. As I wrote there I'm unsure about what to do. Maybe the best is to keep the current checkstyle config

Re: buildbot failure in on ofbizTrunkFrameworkPlugins

2021-05-25 Thread Nicolas Malin
:) thanks for the look Nicolas On 25/05/2021 12:42, Jacques Le Roux wrote: > Actually I'm working on > https://github.com/checkstyle/checkstyle/issues/10012 > > I did not pull. It's now fixed. > > Le 25/05/2021 à 11:33, Jacques Le Roux a écrit : >> Hi Nicolas, >> >> Same here, checking that...

Re: buildbot failure in on ofbizTrunkFrameworkPlugins

2021-05-25 Thread Jacques Le Roux
Actually I'm working on https://github.com/checkstyle/checkstyle/issues/10012 I did not pull. It's now fixed. Le 25/05/2021 à 11:33, Jacques Le Roux a écrit : Hi Nicolas, Same here, checking that... Jacques Le 25/05/2021 à 09:56, Nicolas Malin a écrit : Hi Jacques, It's strange, I haven't

Re: buildbot failure in on ofbizTrunkFrameworkPlugins

2021-05-25 Thread Jacques Le Roux
Hi Nicolas, Same here, checking that... Jacques Le 25/05/2021 à 09:56, Nicolas Malin a écrit : Hi Jacques, It's strange, I haven't this issue on my local Nicolas On 22/05/2021 16:53, Jacques Le Roux wrote: Hi Nicolas, There is a checkstyle issue in this commit. It's not a big deal in

Re: buildbot failure in on ofbizTrunkFrameworkPlugins

2021-05-25 Thread Nicolas Malin
Hi Jacques, It's strange, I haven't this issue on my local Nicolas On 22/05/2021 16:53, Jacques Le Roux wrote: > Hi Nicolas, > > There is a checkstyle issue in this commit. It's not a big deal in > itself but it might hide some other issues coming from now > > TIA to fix it :) > > Jacques > >