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

Unified Diff: releaseAutomation.py

Issue 29756673: Noissue - Adapt best practices for trailing commas (buildtools) (Closed)
Patch Set: Re-run script on Python 2, added flake8-commas extension Created April 19, 2018, 2: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 | « packagerEdge.py ('k') | tests/test_packagerEdge.py » ('j') | tox.ini » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: releaseAutomation.py
===================================================================
--- a/releaseAutomation.py
+++ b/releaseAutomation.py
@@ -185,7 +185,7 @@
rawMetadata = fp.read()
rawMetadata = re.sub(
r'^(\s*version\s*=\s*).*', r'\g<1>%s' % version,
- rawMetadata, flags=re.I | re.M
+ rawMetadata, flags=re.I | re.M,
)
fp.seek(0)
@@ -206,7 +206,7 @@
build_path = os.path.join(
target_path,
getDefaultFileName(metadata, version,
- get_extension(platform, key_file is not None))
+ get_extension(platform, key_file is not None)),
)
packager.createBuild(base_dir, type=platform, outFile=build_path,
@@ -270,7 +270,7 @@
downloads.append(
create_build(platform, baseDir, downloads_repo, version,
- used_key_file)
+ used_key_file),
)
# Only create one commit, one tag and one source archive for all
« no previous file with comments | « packagerEdge.py ('k') | tests/test_packagerEdge.py » ('j') | tox.ini » ('J')

Powered by Google App Engine
This is Rietveld