← Back to team overview

wordpress-charmers team mailing list archive

[Merge] ~mthaddon/charm-k8s-wordpress/+git/print-plugin-names:master into ~wordpress-charmers/charm-k8s-wordpress/+git/wordpress-k8s-image-builder:master

 

Tom Haddon has proposed merging ~mthaddon/charm-k8s-wordpress/+git/print-plugin-names:master into ~wordpress-charmers/charm-k8s-wordpress/+git/wordpress-k8s-image-builder:master.

Commit message:
Print the names of plugins as we download them



Requested reviews:
  Wordpress Charmers (wordpress-charmers)

For more details, see:
https://code.launchpad.net/~mthaddon/charm-k8s-wordpress/+git/print-plugin-names/+merge/377639

Print the names of plugins as we download them

-- 
Your team Wordpress Charmers is requested to review the proposed merge of ~mthaddon/charm-k8s-wordpress/+git/print-plugin-names:master into ~wordpress-charmers/charm-k8s-wordpress/+git/wordpress-k8s-image-builder:master.
diff --git a/fetcher.py b/fetcher.py
index 0d30cc0..9f93073 100755
--- a/fetcher.py
+++ b/fetcher.py
@@ -74,7 +74,7 @@ def get_plugins(zip_plugins, branch_plugins):
     current_zip = 0
     for zip_plugin in zip_plugins:
         current_zip = current_zip + 1
-        print('Downloading zipped plugin {} of {}...'.format(current_zip, total_zips))
+        print('Downloading {} of {} zipped plugins: {} ...'.format(current_zip, total_zips, zip_plugin))
         url = 'https://downloads.wordpress.org/plugin/{}.latest-stable.zip'.format(zip_plugin)
         file_name = os.path.join(os.getcwd(), 'files/plugins', os.path.basename(url))
         with urllib.request.urlopen(url) as response, open(file_name, 'wb') as out_file:
@@ -87,7 +87,7 @@ def get_plugins(zip_plugins, branch_plugins):
     current_branch = 0
     for branch_plugin in branch_plugins:
         current_branch = current_branch + 1
-        print('Downloading branched plugin {} of {}...'.format(current_branch, total_branches))
+        print('Downloading {} of {} branched plugins: {} ...'.format(current_branch, total_branches, branch_plugin))
         url = branch_plugins[branch_plugin].get('url')
         basename = os.path.basename(url)
         if basename.startswith('lp:'):

Follow ups