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

Unified Diff: packagerEdge.py

Issue 29357701: Issue 4548 - Split out Safari and add Edge release automation (Closed)
Patch Set: Avoid linting errors Created Oct. 26, 2016, 3:16 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 | « build.py ('k') | releaseAutomation.py » ('j') | releaseAutomation.py » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: packagerEdge.py
diff --git a/packagerEdge.py b/packagerEdge.py
index c63064e8fbefaaec4482fbcfd2808d89e0a0668e..e2a822869daf9c19c794ace3ea6fb95a4f56123e 100644
--- a/packagerEdge.py
+++ b/packagerEdge.py
@@ -11,6 +11,8 @@ import zipfile
import packager
import packagerChrome
+# These functions are unused here, but releaseAutomation.py expects them.
+from packager import readMetadata, getDefaultFileName # flake8: noqa
# Files and directories expected inside of the .APPX archive.
MANIFEST = 'AppxManifest.xml'
@@ -93,7 +95,7 @@ def create_appx_manifest(params, files, release_build=False):
for size in ['44', '50', '150']:
path = '{}/logo_{}.png'.format(ASSETS_DIR, size)
if path not in files:
- raise KeyError(path + 'is not found in files')
+ raise KeyError(path + ' is not found in files')
params['logo_' + size] = path.replace('/', '\\')
template = _get_template_for(MANIFEST)
« no previous file with comments | « build.py ('k') | releaseAutomation.py » ('j') | releaseAutomation.py » ('J')

Powered by Google App Engine
This is Rietveld