Noissue - Adapt best practices for trailing commas (abpssembly)

sitescripts/extensions/bin/legacy/packagerSafari.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 collections
 import ConfigParser
 import json
 import os
 import re
@@ skipping 74 matching lines @@
         homepage=get_optional('general', 'homepage'),
         updateURL=get_optional('general', 'updateURL'),
         allowedDomains=allowedDomains,
         allowAllDomains=allowAllDomains,
         allowSecurePages=allowSecurePages,
         startScripts=(get_optional('contentScripts', 'document_start') or '').split(),
         endScripts=(get_optional('contentScripts', 'document_end') or '').split(),
         menus=parse_section('menus', 2),
         toolbarItems=parse_section('toolbar_items'),
         popovers=parse_section('popovers'),
-        developerIdentifier=params.get('developerIdentifier')
+        developerIdentifier=params.get('developerIdentifier'),
     ).encode('utf-8')
 
 
 def createInfoModule(params):
     template = getTemplate('safariInfo.js.tmpl')
     return template.render(params).encode('utf-8')
 
 
 def _get_sequence(data):
     from Crypto.Util import asn1
@@ skipping 47 matching lines @@
             module_name = os.path.splitext(os.path.basename(input_filename))[0]
             prefix = os.path.basename(os.path.dirname(input_filename))
             if prefix != 'lib':
                 module_name = '{}_{}'.format(prefix, module_name)
             with open(os.path.join(base_dir, input_filename), 'r') as file:
                 modules.append((module_name, file.read().decode('utf-8')))
             files.pop(input_filename, None)
 
         files[filename] = template.render(
             args=current_args,
-            modules=modules
+            modules=modules,
         ).encode('utf-8')
 
 
 def createBuild(baseDir, type, outFile=None, buildNum=None, releaseBuild=False, keyFile=None, devenv=False):
     metadata = readMetadata(baseDir, type)
     version = getBuildVersion(baseDir, metadata, releaseBuild, buildNum)
 
     if not outFile:
         outFile = getDefaultFileName(metadata, version, 'safariextz' if keyFile else 'zip')
 
@@ skipping 11 matching lines @@
                   process=lambda path, data: processFile(path, data, params))
     files.readMappedFiles(mapped)
     files.read(baseDir, skip=[opt for opt, _ in mapped])
 
     if metadata.has_section('convert_js'):
         convertJS(params, files)
 
     if metadata.has_section('preprocess'):
         files.preprocess(
             [f for f, _ in metadata.items('preprocess')],
-            {'needsExt': True}
+            {'needsExt': True},
         )
 
     if metadata.has_section('import_locales'):
         import_locales(params, files)
 
     if metadata.has_option('general', 'testScripts'):
         files['qunit/index.html'] = createScriptPage(params, 'testIndex.html.tmpl',
                                                      ('general', 'testScripts'))
 
     if keyFile:
         from buildtools import xarfile
         certs, key = xarfile.read_certificates_and_key(keyFile)
         params['developerIdentifier'] = get_developer_identifier(certs)
 
     files['lib/info.js'] = createInfoModule(params)
     files['background.html'] = createScriptPage(params, 'background.html.tmpl',
                                                 ('general', 'backgroundScripts'))
     files['Info.plist'] = createManifest(params, files)
 
     dirname = metadata.get('general', 'basename') + '.safariextension'
     for filename in files.keys():
         files[os.path.join(dirname, filename)] = files.pop(filename)
 
     if not devenv and keyFile:
         from buildtools import xarfile
         xarfile.create(outFile, files, keyFile)
     else:
         files.zip(outFile)