Hi Taher, I have added a comment in Jira, please have a look.
Thanks, Jacopo On Mon, May 30, 2016 at 5:31 PM, Taher Alkhateeb <slidingfilame...@gmail.com > wrote: > Hey Everyone, > > > > My patch has been sitting in OFBIZ-6783 for a few days and I don’t want it > to get outdated as it touches important files including build.xml > > > > This patch shaves off 701 lines of code from OFBiz putting it on a good > healthy diet. I’ve tested it and all is good from my side, I hesitate to > commit before I get one or two feedbacks. Thank you in advance for your > help. > > > > Taher Alkhateeb > > > > From: Taher Alkhateeb [mailto:slidingfilame...@gmail.com] > Sent: 27 May 2016 11:49 > To: OFBIZ Development Mailing List > Subject: Re: proposal to remove cobertura and sonar from ofbiz > > > > Hello Everyone, > > Thank you all for your votes and support. I will submit a patch in > https://issues.apache.org/jira/browse/OFBIZ-6783 and refer to this thread > accordingly. > > I really appreciate your support. Also thank you Adam for the tips, I did > a little bit of research and it seems that yes cobertura is the best > coverage tool to use with ofbiz and I am definitely interested in > reintroducing it in the future. > > I won't commit anything to give time for everyone to test and/or > contribute more to this thread. > > Cheers > > Taher Alkhateeb > > > > On Fri, May 27, 2016 at 10:00 AM, Sharan-F <sharan.f...@gmail.com <mailto: > sharan.f...@gmail.com> > wrote: > > +1 to go ahead too. > > If we have had these issues hanging around for years without being resolved > then I think a clean start sounds the best. > > Thanks > Sharan > > > > -- > View this message in context: > http://ofbiz.135035.n4.nabble.com/proposal-to-remove-cobertura-and-sonar-from-ofbiz-tp4681662p4681716.html > Sent from the OFBiz - Dev mailing list archive at Nabble.com. > > > >
