Hey!
I ran Cookstyle 5.22.6 against this repo and here are the results.
This repo was selected due to the topics of chef-cookbook
Changes
Issues found and resolved with libraries/helpers.rb
8:5 warning: Lint/UnneededCopDisableDirective - Unnecessary disabling of Style/ModuleFunction.
Issues found and resolved with resources/config.rb
6:1 refactor: ChefStyle/TrueClassFalseClassResourceProperties - When setting the allowed types for a resource to accept either true or false values it's much simpler to use true and false instead of TrueClass and FalseClass.
Issues found and resolved with resources/log.rb
4:1 refactor: ChefStyle/TrueClassFalseClassResourceProperties - When setting the allowed types for a resource to accept either true or false values it's much simpler to use true and false instead of TrueClass and FalseClass.
Issues found and resolved with resources/varnish_repo_debian.rb
7:1 refactor: ChefStyle/TrueClassFalseClassResourceProperties - When setting the allowed types for a resource to accept either true or false values it's much simpler to use true and false instead of TrueClass and FalseClass.
14:5 refactor: ChefRedundantCode/AptRepositoryDistributionDefault - There is no need to pass distribution node['lsb']['codename'] to an apt_repository resource as this is done automatically by the apt_repository resource.
Hey! I ran Cookstyle 5.22.6 against this repo and here are the results. This repo was selected due to the topics of chef-cookbook
Changes
Issues found and resolved with libraries/helpers.rb
Lint/UnneededCopDisableDirective
- Unnecessary disabling ofStyle/ModuleFunction
.Issues found and resolved with resources/config.rb
ChefStyle/TrueClassFalseClassResourceProperties
- When setting the allowed types for a resource to accept either true or false values it's much simpler to use true and false instead of TrueClass and FalseClass.Issues found and resolved with resources/log.rb
ChefStyle/TrueClassFalseClassResourceProperties
- When setting the allowed types for a resource to accept either true or false values it's much simpler to use true and false instead of TrueClass and FalseClass.Issues found and resolved with resources/varnish_repo_debian.rb
ChefStyle/TrueClassFalseClassResourceProperties
- When setting the allowed types for a resource to accept either true or false values it's much simpler to use true and false instead of TrueClass and FalseClass.ChefRedundantCode/AptRepositoryDistributionDefault
- There is no need to passdistribution node['lsb']['codename']
to an apt_repository resource as this is done automatically by the apt_repository resource.