On Thu, Apr 28, 2011 at 15:18, Nick Lewis <[email protected]> wrote: > On Thu, Apr 28, 2011 at 8:01 PM, Daniel Pittman <[email protected]> wrote: >> - option("--render-as FORMAT") do |arg| >> - @render_as = arg.to_sym >> + option("--render-as FORMAT") do |_arg| >> + format = _arg.to_sym >> + unless @render_method = Puppet::Network::FormatHandler.format(format) >> + unless format == :for_humans or format == :json >> + raise ArgumentError, "I don't know how to render '#{format}'" >> + end >> + end >> + @render_as = format >> end >> > > Out of curiosity, what's the reason for the rename from arg to _arg here?
Because I confused it with the version that had been turned into a symbol, once, and I forgot to change it back. :) Daniel -- ⎋ Puppet Labs Developer – http://puppetlabs.com ✉ Daniel Pittman <[email protected]> ✆ Contact me via gtalk, email, or phone: +1 (877) 575-9775 ♲ Made with 100 percent post-consumer electrons -- 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.
