Noissue - Adapt best practices for trailing commas (cms)

cms/sources.py

 # This file is part of the Adblock Plus web scripts,
 # Copyright (C) 2006-present eyeo GmbH
 #
 # Adblock Plus is free software: you can redistribute it and/or modify
 # it under the terms of the GNU General Public License version 3 as
 # published by the Free Software Foundation.
 #
 # Adblock Plus is distributed in the hope that it will be useful,
 # but WITHOUT ANY WARRANTY; without even the implied warranty of
 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
@@ skipping 97 matching lines @@
     #
     # Localizable files helpers
     #
 
     @staticmethod
     def localizable_file_filename(locale, filename):
         return 'locales/%s/%s' % (locale, filename)
 
     def list_localizable_files(self):
         default_locale = self.read_config().get('general', 'defaultlocale')
-        return filter(
-            lambda f: os.path.splitext(f)[1].lower() != '.json',
-            self.list_files('locales/%s' % default_locale)

[Sebastian Noack, 2018/04/19 13:51:19]

For whatever reason, my script missed this one.

-        )
+        return filter(lambda f: os.path.splitext(f)[1].lower() != '.json',
+                      self.list_files('locales/%s' % default_locale))
 
     def has_localizable_file(self, locale, filename):
         return self.has_file(self.localizable_file_filename(locale, filename))
 
     def read_localizable_file(self, locale, filename):
         return self.read_file(self.localizable_file_filename(locale, filename), binary=True)[0]
 
     #
     # Static file helpers
     #
@@ skipping 318 matching lines @@
             'resolve_link',
             'read_config',
             'read_template',
             'read_locale',
             'read_include',
             'exec_file',
         ]:
             setattr(source, fname, _memoize(getattr(source, fname)))
 
     return source