Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: sitescripts/extensions/bin/updateRecommendations.py

Issue 29756624: Noissue - Adapt best practices for trailing commas (sitescripts) (Closed)
Patch Set: Re-run script on Python 2, added flake8-commas extension Created April 19, 2018, 2:35 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « sitescripts/extensions/bin/createNightlies.py ('k') | sitescripts/extensions/bin/updateUpdateManifests.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: sitescripts/extensions/bin/updateRecommendations.py
===================================================================
--- a/sitescripts/extensions/bin/updateRecommendations.py
+++ b/sitescripts/extensions/bin/updateRecommendations.py
@@ -27,19 +27,19 @@
tempdir = mkdtemp(prefix='adblockplus')
try:
subprocess.check_call([
- 'hg', 'clone', '-q', '-U', repository, tempdir
+ 'hg', 'clone', '-q', '-U', repository, tempdir,
])
subprocess.check_call([
- 'hg', 'up', '-q', '-R', tempdir, '-r', 'master'
+ 'hg', 'up', '-q', '-R', tempdir, '-r', 'master',
])
writeSubscriptions('recommendations', os.path.join(tempdir, *path))
if subprocess.check_output(['hg', 'stat', '-R', tempdir]) != '':
subprocess.check_call([
'hg', 'commit', '-q', '-R', tempdir, '-u', 'hgbot',
- '-m', 'Noissue - Updated list of recommended subscriptions'
+ '-m', 'Noissue - Updated list of recommended subscriptions',
])
subprocess.check_call([
- 'hg', 'push', '-q', '-R', tempdir, '-r', 'master'
+ 'hg', 'push', '-q', '-R', tempdir, '-r', 'master',
])
finally:
rmtree(tempdir)
« no previous file with comments | « sitescripts/extensions/bin/createNightlies.py ('k') | sitescripts/extensions/bin/updateUpdateManifests.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld