wordpress-charmers team mailing list archive
-
wordpress-charmers team
-
Mailing list archive
-
Message #00026
[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