[prev in list] [next in list] [prev in thread] [next in thread] 

List:       subversion-users
Subject:    Re: Code Reviewing
From:       David Weintraub <qazwart () gmail ! com>
Date:       2009-01-30 14:12:45
Message-ID: e18234670901300612p12f8d971w158de00c6a5af033 () mail ! gmail ! com
[Download RAW message or body]

We have added Findbugs <http://findbugs.sourceforge.net/> which
integrates into Hudson <https://hudson.dev.java.net/>.

The problem is our legacy code. When we first started using Findbugs,
it started off with a lot of energy, but recently Findbugs has been
missing work. When Findbugs does come in, he reeks of alcohol and is
quite beligerent. The other day, I walked by the build machine and
heard Findbugs crying, "I just can't hack it. I just can't hack it."

Findbugs works great with the newer stuff we're doing in Maven, but
our Legacy code has years of awful design and layers of strange coding
practices, and this is where we really need code reviews. It is also
where we have the biggest teams working on it and the most bad habits.

On Thu, Jan 29, 2009 at 4:28 PM, Bob Archer <Bob.Archer@amsi.com> wrote:
>> Thanks. It looks really nice. I think the main problem we have here is
>> that we are relying on code reviews as a way of taking care of poor
>> programming practices.
>
> You could add static code analysis tool like FxCop to your build system.
> This will be an automated "code review" that can catch a lot of bad
> practices bad naming conventions etc. Your devs should run it before
> check-in because they know any FxCop violations will break the build.
>
> BOb
>



-- 
--
David Weintraub
qazwart@gmail.com

------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=1065&dsMessageId=1074636

To unsubscribe from this discussion, e-mail: [users-unsubscribe@subversion.tigris.org].
[prev in list] [next in list] [prev in thread] [next in thread]