Le 20/12/2013 16:03, Felix Frank a écrit :
On 12/20/2013 03:41 PM, DEGREMONT Aurelien wrote:
Assuming this patch is clean/adapted, is this something Puppet could
accept? or there is still other problems?
I can't really give a well founded comment on this.

I like Jeff's approach in spirit, but as far as execution is concerned,
my gut says that something more object oriented would be preferable.
What I'm saying is, instead of enriching interfaces with values that
need passing, an approach based on inheritance and method overrides
might lead to better maintainability later on.
Agreed, but I think guidance from Puppet developers is needed for that.

As far as I'm concern, I do not think I know the puppet code well enough to do this alone.


Aurélien

--
You received this message because you are subscribed to the Google Groups "Puppet 
Developers" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to puppet-dev+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/puppet-dev/52B45E67.2010709%40cea.fr.
For more options, visit https://groups.google.com/groups/opt_out.

Reply via email to