Automated PR: Cookstyle Changes - 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

Automated PR: Cookstyle Changes #558

Closed xorimabot closed 4 years ago

xorimabot commented 4 years ago

Hey! I ran Cookstyle 6.2.9 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/consul_acl.rb

109:11 convention: Style/RedundantReturn - Redundant return detected.

Issues found and resolved with libraries/consul_policy.rb

122:11 convention: Style/RedundantReturn - Redundant return detected.

Issues found and resolved with libraries/consul_role.rb

119:11 convention: Style/RedundantReturn - Redundant return detected.

Issues found and resolved with libraries/consul_token.rb

141:11 convention: Style/RedundantReturn - Redundant return detected.

Issues found and resolved with libraries/helpers.rb

19:7 refactor: ChefStyle/UnnecessaryOSCheck - Use the platform_family?() helpers instead of node['os] == 'foo' for platform_families that match 1:1 with OS values.

kitchen-porter commented 4 years ago

	1 Warning
:warning:	This Pull Request is probably missing tests.

Generated by :no_entry_sign: Danger