Remove dead code - Githubissues

sous-chefs / consul

Development repository for the consul cookbook

https://supermarket.chef.io/cookbooks/consul

Apache License 2.0

192 stars 244 forks source link

Closed legal90 closed 7 years ago

legal90 commented 7 years ago

"chef-sugar" - this dependency was removed in #387, but include_recipe worked fine only because it is included by "firewall::default". We still don't need it, so it could be removed.
"version" attribute is not defined in "consul_service" resource, it is not used and should not be passed there. Without chef-sugar it bombs this way:

       NoMethodError
       -------------
       undefined method `version' for ConsulCookbook::Resource::ConsulService

codecov-io commented 7 years ago

No coverage report found for master at c639a3e.

Powered by Codecov. Last update c639a3e...a640bad

lock[bot] commented 4 years ago

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.