mahout-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Ted Dunning <>
Subject Re: Turn on code inspections, please
Date Mon, 11 Jun 2012 14:20:29 GMT
If you mean eclipse integration with the style check, go ahead.  Something is better than nothing.

Sent from my iPhone

On Jun 11, 2012, at 4:39 AM, Benson Margulies <> wrote:

> It may take me a few days.
> How much do people care about Eclipse integration? Can I ignore that
> for the moment?
> On Sun, Jun 10, 2012 at 10:58 PM, Drew Farris <> wrote:
>> Benson, this sounds promising. Will you post a patch that turns these
>> on so we can take it out for a spin?
>> I think committers should not be checking in code that doesn't conform
>> to the project's standards.
>> Drew
>> On Sun, Jun 10, 2012 at 10:09 PM, Benson Margulies
>> <> wrote:
>>> I hesitate to remind you all that the maven plugins can be wired up
>>> for at least checkstyle and PMD as parts of the build that *fail*, not
>>> just report, and that several other Apache projects live very happily
>>> this way. This makes it pretty nearly impossible to check in code that
>>> doesn't meet whatever standards are configured.
>>> On Sun, Jun 10, 2012 at 5:19 PM, Robin Anil <> wrote:
>>>> Grant, you mentioned you have some documented steps to hookup Jira patch
>>>> submit with jenkins. Can you share those. Findbugs/Checkstyle/Pmd/Clover
>>>> already integrated in our Jenkins build. I bet we should be able to get
>>>> decent stats on each patch. To me that's a more sustainable process after
>>>> doing a one time massive fix.
>>>> Robin
>>>> On Sun, Jun 10, 2012 at 12:03 PM, Grant Ingersoll <>wrote:
>>>>> On Jun 9, 2012, at 7:17 PM, Sean Owen wrote:
>>>>>> Guys, I'm preparing a large new patch that fixes style problems in
>>>>>> code, for after the code freeze. This is my last pass at this for
>>>>>> Mahout.
>>>>>> Style is not a big deal, though it's probably not good that random
>>>>>> non-standard Java is committed to the project. The only hard 'fix'
>>>>>> this long-standing phenomenon is requiring a review process, and
>>>>>> is too much. I don't think this project adheres to standards so much,
>>>>>> and such is life.
>>>>> Perhaps we should at least clean up style before every release.  I've
>>>>> other projects do this and while it isn't perfect, it does mean that
>>>>> start from a clean slate every time.
>>>>> Naturally, committers can also stylize right before committing, too.
>>>>> usually reduces the burden on the contributor, but keeps the code base
>>>>> good form.
>>>>>> However, simply turning on code inspections in a modern IDE like
>>>>>> IntelilJ is turning up plain bugs in the code. I want to call out
>>>>>> few, because I want to fix them (after 0.7), but also because I want
>>>>>> to make the point that static analysis can find bugs. Because it
>>>>>> it should. I think open source projects can and should be the finest
>>>>>> output of the best and brightest. And at "mere" Google, stuff that
>>>>>> static analysis finds would never have gotten to even code review.
>>>>>> Hence I am somewhat dismayed to see so many problems being committed
>>>>>> without review into the code base.
>>>>> +1.
>>>>> -Grant

View raw message