On May 7, 2012, at 9:30 AM, Philip Brown wrote:

> 
> 
> On Saturday, May 5, 2012 11:44:42 PM UTC-7, Luke Kanies wrote:
> On May 5, 2012, at 8:21 PM, Philip Brown wrote:
>> This stuff may as well be black magic. it reaaally needs to be added to the 
>> "how to write your own type" documentation!! :(
>> It is not at all apparent, than a single line,
>> "ensurable" is some short cut for an expandable "property definition"
>> 
>> Similarly for the "defaultvalues" line. not particular understandable, but  
>> "it just has to be there".
>> 
>> (At least I have C++/java experience, so I can guess how it works. but for 
>> someone else trying to learn how to write their own types, etc. .. yuck)
> 
> I largely agree - for the very simple cases of the very complex cases, it 
> works well enough, but for anything in between it's a bit rough.
> 
> I think the RAL is in drastic need of refactoring, but it's a large job with 
> a larger backward compatibility problem and isn't directly necessary to get 
> anything else done, so it never quite gets prioritized like I'd like.
> 
> 
> 
> Refactoring it is overly drastic. I think that improved documentation would 
> mostly alleviate the issue.

Good point.   I know the warts too well. :)

Can you open tickets for that lack of documentation?

-- 
Luke Kanies | http://about.me/lak | http://puppetlabs.com/ | +1-615-594-8199

-- 
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.

Reply via email to