diff options
| author | Tim Smith <tsmith84@gmail.com> | 2020-07-07 15:19:21 -0700 |
|---|---|---|
| committer | Tim Smith <tsmith84@gmail.com> | 2020-07-07 15:19:21 -0700 |
| commit | 1485a862053fb20ade1f93bb82c6918cd4078975 (patch) | |
| tree | 21ae9d81524f56a4b52449424550259422b1b9a5 /lib/chef/cookbook/remote_file_vendor.rb | |
| parent | 3e9e6d41a31bd1ac932f7f4100102466b4fdfe05 (diff) | |
| download | chef-1485a862053fb20ade1f93bb82c6918cd4078975.tar.gz | |
Avoid assigning variables before returning if we don't have to
These seemed pretty straight forward. There's a new Rubocop cop for these.
Signed-off-by: Tim Smith <tsmith@chef.io>
Diffstat (limited to 'lib/chef/cookbook/remote_file_vendor.rb')
| -rw-r--r-- | lib/chef/cookbook/remote_file_vendor.rb | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/lib/chef/cookbook/remote_file_vendor.rb b/lib/chef/cookbook/remote_file_vendor.rb index e2a9798c8a..a6a6e96e06 100644 --- a/lib/chef/cookbook/remote_file_vendor.rb +++ b/lib/chef/cookbook/remote_file_vendor.rb @@ -72,10 +72,8 @@ class Chef Chef::Log.trace("Current checksum: #{current_checksum}; manifest checksum: #{found_manifest_record["checksum"]})") end - full_path_cache_filename = Chef::FileCache.load(cache_filename, false) - # return the filename, not the contents (second argument= false) - full_path_cache_filename + Chef::FileCache.load(cache_filename, false) end def validate_cached_copy(cache_filename) |