directory-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Felix Knecht <fel...@apache.org>
Subject Re: Shared file review
Date Mon, 06 Sep 2010 18:23:58 GMT
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On 09/06/10 09:42, Emmanuel Lecharny wrote:
>  Hi guys,
> 
> as I was just back from hollidays, and as my brain was fried, I started
> with a simple task this week-end : reviewing shared files. I have
> created a file I put in shared (file-review.txt) which track the review
> progress.
> 
> I'm checking many aspects :
> - ASF header is present
> - Class header is present and correct (javadoc, @author, @pram for
> generic classes)
> - Fields Javadoc
> - Methods Javadoc
> - No more TODO (TODOs must be transformed to JIRAs)
> - No more error message without i18n, explicit error code (ie, not
> ERR_02008 for instance)
> - No more warnings
> - Formater applied
> - Checkstyle and findbugs cleared
> 
> About Checkstyle and Findbugs, I have cleared a lot of items, some of
> them will remain, so may be we can tune the config to get rid of them :
> 'xxx' hide a field (this is because we use the same name in the set(
> param ) methods than the associated field.
> 
> Felix, may be you can tune that ?

NP I'll do for the files in file-review.txt.

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2.0.16 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/

iEYEARECAAYFAkyFMb4ACgkQ2lZVCB08qHEEpQCfWEoB17x+DXH30bdQzFmhNYLz
Z2gAoN8I0Jv1b0XAwP/od7t/ijEKQyTd
=dDZu
-----END PGP SIGNATURE-----

Mime
View raw message