Célestin Matte [off-list ref] writes:
Subject: [PATCH 01/18] Follow perlcritic's recommendations - level 5 and 4
It would be better to prefix commit messages with "git-remote-mediawiki: ".
Fix warnings from perlcritic's level 5 and 4.
It would be cool to have a "make perlcritic" target in the Makefile so
that future developers can easily re-run it and avoid repeating the same
mistakes. As much as possible, "make perlcritic" should produce no
output at the end of your patch series (either the warnings should be
fixed, or they should be disabled).
--
Matthieu Moy
http://www-verimag.imag.fr/~moy/
Le 07/06/2013 10:10, Matthieu Moy a écrit :
It would be cool to have a "make perlcritic" target in the Makefile so
that future developers can easily re-run it and avoid repeating the same
mistakes. As much as possible, "make perlcritic" should produce no
output at the end of your patch series (either the warnings should be
fixed, or they should be disabled).
The problem is that I took some policies into account for some parts of
the code, but not for all of it. For instance, in commit [15/18], I put
some numeric values in constants, but not all of them, as I think having
"arg[3]" in the code does make sense. Ignoring this policy for future
developement just to prevent the related warnings from appearing would
prevent us to see useful warning messages from this policy.
Therefore, there still are a dozen warning messages appearing in the
current state...
Anyway, should this be integrated in the current patch or added as an
independant patch?
--
Célestin Matte