Hi all, Do I need to modify something to allow the PR be accepted?
Thank Antonio 2016-03-17 14:45 GMT+01:00 Antonio Gomes Rodrigues <ra0...@gmail.com>: > Hi, > > Unitary tests fixed > > Antonio > > Cet e-mail a été envoyé depuis un ordinateur protégé par Avast. > www.avast.com > <https://www.avast.com/fr-fr/lp-esg-fav?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=webmail&utm_term=OA-2109-B> > <#6858209084203675670_DDB4FAA8-2DD7-40BB-A1B8-4E2AA1F9FDF2> > > 2016-03-17 12:38 GMT+01:00 sebb <seb...@gmail.com>: > >> On 17 March 2016 at 11:20, Antonio Gomes Rodrigues <ra0...@gmail.com> >> wrote: >> > I will fix the unit test asap >> > >> > Can you provide the checkstyle configuration to run it in local before >> > commit the change? >> >> It's in SVN. >> >> But as already noted it does not catch everything. >> >> The best is to follow the layout convention for the file that you are >> working on. >> This may vary between files (and certainly does for source file types). >> >> > Antonio >> > >> > Cet e-mail a été envoyé depuis un ordinateur protégé par Avast. >> > www.avast.com >> > < >> https://www.avast.com/fr-fr/lp-esg-fav?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=webmail&utm_term=OA-2109-B >> > >> > <#DDB4FAA8-2DD7-40BB-A1B8-4E2AA1F9FDF2> >> > >> > 2016-03-17 12:15 GMT+01:00 Vladimir Sitnikov < >> sitnikov.vladi...@gmail.com>: >> > >> >> Antonio>Do we have a checkstyle configuration or something like that >> >> to avoid waste >> >> Antonio>time (your time and mine) in how to code to JMeter? >> >> >> >> There's a checkstyle (it is run as a part of Travis job), however it >> >> would never check 100% of the issues. >> >> My "else branch swap" request was just to make sure the diff does not >> >> bring unintentional modifications. >> >> >> >> By the way, have you seen that Travis job fails? Any chances your fix >> that? >> >> >> >> Vladimir >> >> >> > >