Fun with App::Prove::Plugin

[Andy Armstrong]

On 7 Feb 2009, at 22:41, Steve Purkis wrote:
>> My preference would be to mate that with the existing -P switch but
>> I'm flexible :)
>
> Agreed.
>
> I've attached a patch for this (+tests+docs) - will commit when I  
> can remember my friggin password.

Sorry for the delay sorting that out - I was away for the weekend.

> I ended up going with a separate Plugin->load() method, rather than  
> changing the args passed to Plugin->import() -- this affects modules  
> loaded with -M too, so I wanted it to be backwards compat.   
> Hopefully it all makes sense; if not lemme know (or fix it ;-).
>
> It might make sense to split the plugin loading from the module  
> loading - as it stands, if any module loaded has a load() method,  
> it'll get called.  If you guys think that's enough of a problem we  
> should change it.
>
> It might also make sense to load the plugins before checking help/ 
> man/version/dry (so the plugins could set any of these if need be).   
> And it might make sense to keep track of the actual plugin classes  
> loaded.  But I'm outta time...


I'm outta time too - but commit it and I'll have a look, thanks ;)

-- 
Andy Armstrong, Hexten