Re: Module proposal: Test::Timer

2007-03-08 Thread Paul LeoNerd Evans
On Thu, Mar 08, 2007 at 07:15:49AM +0100, Jonas B. Nielsen wrote: Well all are included, version 0.01 is available on CPAN. I've had a look, and I have the following comments to make: * You seem to have a mixture of tabs and spaces for indentation, and the tabs seem to be spaced at 4 spaces.

Re: Module proposal: Test::Timer

2007-03-08 Thread Andy Armstrong
On 8 Mar 2007, at 12:53, Paul LeoNerd Evans wrote: * You've put all the POD at the end of the file. I usually find that if I put the POD for each function just above the function code itself, I'm much more likely to keep it updated when I change the code. Makes it easier to find too,

Re: Module proposal: Test::Timer

2007-03-08 Thread Adrian Howard
On 8 Mar 2007, at 13:28, Andy Armstrong wrote: On 8 Mar 2007, at 12:53, Paul LeoNerd Evans wrote: * You've put all the POD at the end of the file. I usually find that if I put the POD for each function just above the function code itself, I'm much more likely to keep it updated when I

Re: per-author META.yml

2007-03-08 Thread David Golden
On 3/7/07, Ken Williams [EMAIL PROTECTED] wrote: In the specific instance we're talking about, cc_author, I wouldn't want it in any such file, whether per-author or per-distribution; I'd want it as a preference I can set within the cpan-testers system. Because if I change my mind and decide I'd