Automated PR: Cookstyle Changes - Githubissues

sous-chefs / jenkins

Development repository for the jenkins cookbook

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

Apache License 2.0

424 stars 635 forks source link

Automated PR: Cookstyle Changes #738

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 cookbook

Changes

Issues found and resolved with libraries/credentials_file.rb

91:33 convention: Style/HashEachMethods - Use each_key instead of keys.each.

Issues found and resolved with libraries/credentials_secret_text.rb

114:33 convention: Style/HashEachMethods - Use each_key instead of keys.each.

Issues found and resolved with libraries/credentials_user.rb

81:33 convention: Style/HashEachMethods - Use each_key instead of keys.each.

Issues found and resolved with libraries/slave.rb

401:33 convention: Style/HashEachMethods - Use each_key instead of keys.each.

Issues found and resolved with libraries/slave_jnlp.rb

64:14 warning: ChefDeprecations/ChefWindowsPlatformHelper - Use platform?('windows') instead of the legacy Chef::Platform.windows? helper.
91:83 warning: ChefDeprecations/ChefWindowsPlatformHelper - Use platform?('windows') instead of the legacy Chef::Platform.windows? helper.
95:17 warning: ChefDeprecations/ChefWindowsPlatformHelper - Use platform?('windows') instead of the legacy Chef::Platform.windows? helper.