diff options
author | Noah Kantrowitz <noah@coderanger.net> | 2017-04-10 16:58:22 -0700 |
---|---|---|
committer | Noah Kantrowitz <noah@coderanger.net> | 2017-04-10 16:58:22 -0700 |
commit | 3997dbf3efb9850342121497aa27f1964a22b394 (patch) | |
tree | 4887489fc95d4d2412f72b54c95fcfe8a55dc415 | |
parent | 42d2a515571e60607a6b9b195d409784105b7e80 (diff) | |
download | chef-3997dbf3efb9850342121497aa27f1964a22b394.tar.gz |
Use each_with_object rather than inject because it’s 1% less footgun.
Signed-off-by: Noah Kantrowitz <noah@coderanger.net>
-rw-r--r-- | lib/chef/cookbook/manifest_v0.rb | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/lib/chef/cookbook/manifest_v0.rb b/lib/chef/cookbook/manifest_v0.rb index 3e50d86071..d818f3072f 100644 --- a/lib/chef/cookbook/manifest_v0.rb +++ b/lib/chef/cookbook/manifest_v0.rb @@ -44,7 +44,7 @@ class Chef result.delete("all_files") files = manifest.by_parent_directory - files.keys.inject(result) do |memo, parent| + files.keys.each_with_object(result) do |parent, memo| if COOKBOOK_SEGMENTS.include?(parent) memo[parent] ||= [] files[parent].each do |file| @@ -53,7 +53,6 @@ class Chef memo[parent] << file end end - memo end result.merge({ "frozen?" => manifest.frozen_version?, "chef_type" => "cookbook_version" }) |