Noissue - Adapt best practices for trailing commas (buildtools)

packagerEdge.py

 # This Source Code Form is subject to the terms of the Mozilla Public
 # License, v. 2.0. If a copy of the MPL was not distributed with this
 # file, You can obtain one at http://mozilla.org/MPL/2.0/.
 
 import base64
 import hashlib
 import json
 import mimetypes
 import os
 import zipfile
@@ skipping 29 matching lines @@
 
 def _make_blockmap_entry(filename, data):
     blocks = [data[i:i + BLOCKSIZE] for i in range(0, len(data), BLOCKSIZE)]
     return {
         'name': filename.replace('/', '\\'),
         'size': len(data),
         'lfh_size': _lfh_size(filename),
         'blocks': [
             {'hash': base64.b64encode(hashlib.sha256(block).digest())}
             for block in blocks
-        ]
+        ],
     }
 
 
 def create_appx_blockmap(files):
     """Create APPX blockmap for the list of files."""
     # We don't support AppxBlockmap.xml generation for compressed zip files at
     # the moment. The only way to reliably calculate the compressed size of
     # each 64k chunk in the zip file is to override the relevant parts of
     # `zipfile` library. We have chosen to not do it so we produce an
     # uncompressed zip file that is later repackaged by Windows Store with
@@ skipping 63 matching lines @@
     names = sorted(files.keys(), key=len, reverse=True)
     for filename in names:
         files['{}/{}'.format(EXTENSION_DIR, filename)] = files.pop(filename)
 
 
 def create_content_types_map(filenames):
     """Create [Content_Types].xml -- a mime type map."""
     params = {'defaults': {}, 'overrides': {}}
     overrides = {
         BLOCKMAP: 'application/vnd.ms-appx.blockmap+xml',
-        MANIFEST: 'application/vnd.ms-appx.manifest+xml'
+        MANIFEST: 'application/vnd.ms-appx.manifest+xml',
     }
     types = mimetypes.MimeTypes()
     types.add_type('application/octet-stream', '.otf')
     for filename in filenames:
         ext = os.path.splitext(filename)[1]
         if ext:
             content_type = types.guess_type(filename, strict=False)[0]
             if content_type is not None:
                 params['defaults'][ext[1:]] = content_type
         if filename in overrides:
@@ skipping 52 matching lines @@
         for name, path in metadata.items('appx_assets'):
             path = os.path.join(baseDir, path)
             files.read(path, '{}/{}'.format(ASSETS_DIR, name))
 
     files[MANIFEST] = create_appx_manifest(params, files,
                                            buildNum, releaseBuild)
     files[BLOCKMAP] = create_appx_blockmap(files)
     files[CONTENT_TYPES] = create_content_types_map(files.keys() + [BLOCKMAP])
 
     files.zip(outfile, compression=zipfile.ZIP_STORED)