Le 05/12/2012 14:31, Thierry Vignaud a écrit :
On 5 December 2012 14:09, Thierry Vignaud <[email protected]> wrote:
initial import

Added Paths

rpm/urpmi/trunk/t/03pod.t

1) we already have t/pod.t
Indeed, I missed it at first glance. It's broken, tough, as it don't find the perl module directory.

2) only the first tests are numbered because others rely on them
Ordering is not strictly needed, but I personnaly prefer to run such kind of dumb generic tests before application-specific tests. Would you be OK with the three pod-related tests renamed as:
pod-syntax.t
pod-coverage.t
pod-spelling.t

3) I'm pretty sure you can get with a better log message such as
explaining the purpose...
"leverage the need of human peer-review on mailing lists for basic pod syntax errors" :) ?

4) tests do not need to be executable
As any other perl program. But in this case, the shell bang in the first line is perfectly useless.

--
BOFH excuse #107:

The keyboard isn't plugged in

Reply via email to