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

Unified Diff: cms/bin/translate.py

Issue 29756692: Noissue - Adapt best practices for trailing commas (cms) (Closed)
Patch Set: Created April 19, 2018, 1:09 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 | « no previous file | cms/converters.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cms/bin/translate.py
===================================================================
--- a/cms/bin/translate.py
+++ b/cms/bin/translate.py
@@ -45,11 +45,11 @@
url = '/api/project/%s/%s?%s' % (
urllib.quote(self.project_name),
urllib.quote(api_endpoint),
- urllib.urlencode((('key', self.api_key),) + query_params)
+ urllib.urlencode((('key', self.api_key),) + query_params),
)
try:
response = self.connection.request(
- request_method, str(url), **kwargs
+ request_method, str(url), **kwargs,
)
except urllib3.exceptions.HTTPError:
logger.error('Connection to API endpoint %s failed', url)
@@ -72,7 +72,7 @@
response = self.raw_request(
request_method, api_endpoint, (('json', '1'),),
- fields=fields, preload_content=False
+ fields=fields, preload_content=False,
)
try:
@@ -148,7 +148,7 @@
logger.info('Enabling the required locales for the Crowdin project...')
crowdin_api.request(
'POST', 'edit-project',
- data={'languages': enabled_locales | required_crowdin_locales}
+ data={'languages': enabled_locales | required_crowdin_locales},
)
return required_locales
@@ -271,7 +271,7 @@
if ext.lower() == '.json' and locale in inverted_required_locales:
output_path = os.path.join(
locale_path, inverted_required_locales[locale],
- *file_path + [file_name]
+ *file_path + [file_name],
)
with archive.open(member) as source_file:
locale_file_contents = json.load(source_file)
« no previous file with comments | « no previous file | cms/converters.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld