On Jul 3, 2011, at 1:57 PM, Daniel Pittman wrote: > On Sun, Jul 3, 2011 at 13:40, James Turnbull <[email protected]> wrote: >> Luke Kanies wrote: >>> >>> One of the cool new things in 2.7 is easy serialization of classes >>> and definitions - meaning you can remotely query a master for which >>> ones exist, what parameters they require, etc. There's even a >>> 'resource_type' face for doing exactly that. >>> >>> The only major problem with this is that it doesn't show any builtin >>> resource types, only the classes, definitions, and nodes from the >>> language. >>> >>> In an attempt to fix this and many other things, I'm currently trying >>> to combine the Puppet::Resource::Type and Puppet::Type classes, and I >>> have a question: >>> >>> When providing the list of acceptable parameters for a given >>> class/type/etc, should we list all of the metaparams (e.g., noop and >>> tag) as valid parameters for all of them, or should we rely on the >>> user to know that? >>> >>> My tendency is to list them all, but what do you think? >> >> We should list them all. This would be the principal of least surprise I >> think. > > Absolutely agree.
Ok. I also realized that I've kinda done this refactor the wrong way round: Instead of doing a bunch of clean-up refactors then the big reorganization, I did the big reorganization which is now driving me to do the many small cleanups. I can't quite tell yet if it's going to be ugly but ok, or a complete waste of time. :/ -- The trouble with the rat race is that even if you win, you're still a rat. -- Lily Tomlin --------------------------------------------------------------------- Luke Kanies | http://puppetlabs.com | http://about.me/lak Join us in PDX for PuppetConf: http://bit.ly/puppetconfsig -- You received this message because you are subscribed to the Google Groups "Puppet Developers" group. To post to this group, send email to [email protected]. To unsubscribe from this group, send email to [email protected]. For more options, visit this group at http://groups.google.com/group/puppet-dev?hl=en.
