Garrett Honeycutt wrote: > > We currently are not doing validation and do not have all the > properties listed. I propose that we accept this patch, since it adds > the rest of the missing properties and create a new ticket to add > validation to the entirety of the properties. This gives will ensure > that we add functionality with this commit and have another ticket > solely for validation of all properties.
Sure but I don't recommend we merge the additional functionality until the validation is in place. Making an existing problem worse when we could easily fix it seems wrong. :) Regards James -- James Turnbull Puppet Labs 1-503-734-8571 -- 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.
