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

Unified Diff: sitescripts/extensions/bin/legacy/packagerSafari.py

Issue 29756646: Noissue - Adapt best practices for trailing commas (abpssembly) (Closed)
Patch Set: Re-run script on Python 2, added flake8-commas extension Created April 19, 2018, 2:41 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/updateRecommendations.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: sitescripts/extensions/bin/legacy/packagerSafari.py
===================================================================
--- a/sitescripts/extensions/bin/legacy/packagerSafari.py
+++ b/sitescripts/extensions/bin/legacy/packagerSafari.py
@@ -92,7 +92,7 @@
menus=parse_section('menus', 2),
toolbarItems=parse_section('toolbar_items'),
popovers=parse_section('popovers'),
- developerIdentifier=params.get('developerIdentifier')
+ developerIdentifier=params.get('developerIdentifier'),
).encode('utf-8')
@@ -160,7 +160,7 @@
files[filename] = template.render(
args=current_args,
- modules=modules
+ modules=modules,
).encode('utf-8')
@@ -192,7 +192,7 @@
if metadata.has_section('preprocess'):
files.preprocess(
[f for f, _ in metadata.items('preprocess')],
- {'needsExt': True}
+ {'needsExt': True},
)
if metadata.has_section('import_locales'):
« no previous file with comments | « sitescripts/extensions/bin/createNightlies.py ('k') | sitescripts/extensions/bin/updateRecommendations.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld