chef-cookbooks / runit

Development repository for the Chef Runit Cookbook
https://supermarket.chef.io/cookbooks/runit
Apache License 2.0
106 stars 197 forks source link

fix warnings due to nil property assignment #183

Closed lamont-granquist closed 8 years ago

lamont-granquist commented 8 years ago

group new_resource.group is a getter when new_resource.group is nil, which is behavior that is going to change in Chef-13.

this suppresses all the recent warning spam from chef.

tas50 commented 8 years ago

@cwjohnston Any chance we can get this merged and released? It will quiet down the SPAM of deprecation warnings people get on Chef 12.7 and the soon to be released 12.8. We have a fix in Chef to quiet the noise, but that won't make it until the next release.

lamont-granquist commented 8 years ago

*next release after 12.8

tas50 commented 8 years ago

^ that

cwjohnston commented 8 years ago

@tas50 I've assigned this to the milestone for next release and will have a look soon.

cwjohnston commented 8 years ago

Looks good, thanks!