Sure, that would simplify it more.  I'll just remove the `if
option.instance_of?(Symbol)` and other case so we handle them both the same


On Thu, May 2, 2013 at 10:11 AM, Trae Robrock <notifications@github.com>wrote:

> In libraries/default.rb:
>
> > @@ -19,7 +19,7 @@
> >
> >  def collectd_key(option)
> >    return option.to_s.split('_').map { |x| x.capitalize }.join() if option.instance_of?(Symbol)
> > -  option
> > +  option.split('_').map { |x| x.capitalize }.join()
>
> This will no longer work for symbols. Could we keep the to_s after option
> to handle both cases?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/miah/chef-collectd/pull/26/files#r4057937>
> .
>