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

Unified Diff: sitescripts/submit_email/web/submit_email.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/stats/test/logprocessor.py ('k') | sitescripts/templateFilters.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: sitescripts/submit_email/web/submit_email.py
===================================================================
--- a/sitescripts/submit_email/web/submit_email.py
+++ b/sitescripts/submit_email/web/submit_email.py
@@ -49,7 +49,7 @@
except ValueError:
return send_simple_response(
start_response, 400,
- 'Please enter a valid email address.'
+ 'Please enter a valid email address.',
)
params = [('email', email), ('signature', sign(config, email)), ('product', product)]
@@ -63,15 +63,15 @@
'recipient': email,
'verification_url': '%s?%s' % (
urljoin(wsgiref.util.application_uri(environ), VERIFICATION_PATH),
- urlencode(params)
- )
- }
+ urlencode(params),
+ ),
+ },
)
return send_simple_response(
start_response, 200,
'A confirmation email has been sent. Please check '
- 'your email and click the confirmation link.'
+ 'your email and click the confirmation link.',
)
@@ -90,7 +90,7 @@
if sign(config, email) != signature:
return send_simple_response(
start_response, 403,
- 'Invalid signature in verification request.'
+ 'Invalid signature in verification request.',
)
with open(filename, 'ab', 0) as file:
« no previous file with comments | « sitescripts/stats/test/logprocessor.py ('k') | sitescripts/templateFilters.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld