Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Commit 2a4f29d

Browse files
Changing whyrun description format into string instead of array.
1 parent c02b226 commit 2a4f29d

File tree

2 files changed

+4
-8
lines changed

2 files changed

+4
-8
lines changed

providers/pip.rb

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -46,8 +46,7 @@ def whyrun_supported?
4646
end
4747

4848
if install_version
49-
description = []
50-
description << "install package #{@new_resource} version #{install_version}"
49+
description = "install package #{@new_resource} version #{install_version}"
5150
converge_by(description) do
5251
Chef::Log.info("Installing #{@new_resource} version #{install_version}")
5352
status = install_package(@new_resource.package_name, install_version, timeout)
@@ -64,8 +63,7 @@ def whyrun_supported?
6463

6564
if @current_resource.version != candidate_version
6665
orig_version = @current_resource.version || "uninstalled"
67-
description = []
68-
description << "upgrade #{@current_resource} version from #{@current_resource.version} to #{candidate_version}"
66+
description = "upgrade #{@current_resource} version from #{@current_resource.version} to #{candidate_version}"
6967
converge_by(description) do
7068
Chef::Log.info("Upgrading #{@new_resource} version from #{orig_version} to #{candidate_version}")
7169
status = upgrade_package(@new_resource.package_name, candidate_version, timeout)
@@ -81,8 +79,7 @@ def whyrun_supported?
8179
end
8280

8381
if removing_package?
84-
description = []
85-
description << "remove package #{@new_resource}"
82+
description = "remove package #{@new_resource}"
8683
converge_by(description) do
8784
Chef::Log.info("Removing #{@new_resource}")
8885
remove_package(@current_resource.package_name, @new_resource.version, timeout)

providers/virtualenv.rb

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -39,8 +39,7 @@ def whyrun_supported?
3939

4040
action :delete do
4141
if exists?
42-
description = []
43-
description << "delete virtualenv #{@new_resource} at #{@new_resource.path}"
42+
description = "delete virtualenv #{@new_resource} at #{@new_resource.path}"
4443
converge_by(description) do
4544
Chef::Log.info("Deleting virtualenv #{@new_resource} at #{@new_resource.path}")
4645
FileUtils.rm_rf(@new_resource.path)

0 commit comments

Comments
 (0)