This is now on my https://github.com/andreacampi/chef/tree/CHEF-3249 branch.I *think* I've incorporated every functional requirement we talked about (except from my "wishlist" item about collections).I didn't get rid of the mixin because it makes testing quite easier, while keeping classes at a reasonable time. We can do it later (or in the future).Also Daniel, I haven't followed your proposal for lib/chef/provider/template/support_class.rb: lib/chef/provider/template would need to require it, but in turn this would need to see a declaration for Chef::Provider::Template or it would result in a bad inheritance chain.Very much solvable, but I want to make sure I follow any relevant "coding standard". I.e. do we use "forward declarations" elsewhere in the tree?Other than that, as far as I'm concerned this is good to go :) I'll wait for more comments and go ahead with the pull request.Andrea
Archive powered by MHonArc 2.6.16.