After mailing the patch Jesse and I came up with an alternative to the regex in:

  text_line :incomplete, :match => Regexp.new( "^(?!\s*" + ( ['\S+'] *
mandatory_fields.length).join('\s+') + ")" )

which is arguably more readable:

   field_pattern = '(\s* (>~\S+))'

    /^(?!#{field_pattern}{#{mandatory_fields.length}})/
-- 
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