mirror of
https://github.com/platformio/platformio-core.git
synced 2025-08-01 10:54:27 +02:00
Fix PyLint for v2.9.3
This commit is contained in:
@@ -15,6 +15,7 @@ disable=
|
|||||||
useless-object-inheritance,
|
useless-object-inheritance,
|
||||||
useless-import-alias,
|
useless-import-alias,
|
||||||
bad-option-value,
|
bad-option-value,
|
||||||
|
consider-using-dict-items,
|
||||||
|
|
||||||
; PY2 Compat
|
; PY2 Compat
|
||||||
super-with-arguments,
|
super-with-arguments,
|
||||||
|
@@ -78,6 +78,7 @@ def remove_unnecessary_core_packages(dry_run=False):
|
|||||||
pkg = pm.get_package(spec)
|
pkg = pm.get_package(spec)
|
||||||
if not pkg:
|
if not pkg:
|
||||||
continue
|
continue
|
||||||
|
# pylint: disable=no-member
|
||||||
best_pkg_versions[pkg.metadata.name] = pkg.metadata.version
|
best_pkg_versions[pkg.metadata.name] = pkg.metadata.version
|
||||||
|
|
||||||
for pkg in pm.get_installed():
|
for pkg in pm.get_installed():
|
||||||
|
@@ -57,7 +57,9 @@ class BaseArchiver(object):
|
|||||||
|
|
||||||
class TARArchiver(BaseArchiver):
|
class TARArchiver(BaseArchiver):
|
||||||
def __init__(self, archpath):
|
def __init__(self, archpath):
|
||||||
super(TARArchiver, self).__init__(tarfile_open(archpath))
|
super(TARArchiver, self).__init__(
|
||||||
|
tarfile_open(archpath) # pylint: disable=consider-using-with
|
||||||
|
)
|
||||||
|
|
||||||
def get_items(self):
|
def get_items(self):
|
||||||
return self._afo.getmembers()
|
return self._afo.getmembers()
|
||||||
@@ -99,7 +101,9 @@ class TARArchiver(BaseArchiver):
|
|||||||
|
|
||||||
class ZIPArchiver(BaseArchiver):
|
class ZIPArchiver(BaseArchiver):
|
||||||
def __init__(self, archpath):
|
def __init__(self, archpath):
|
||||||
super(ZIPArchiver, self).__init__(ZipFile(archpath))
|
super(ZIPArchiver, self).__init__(
|
||||||
|
ZipFile(archpath) # pylint: disable=consider-using-with
|
||||||
|
)
|
||||||
|
|
||||||
@staticmethod
|
@staticmethod
|
||||||
def preserve_permissions(item, dest_dir):
|
def preserve_permissions(item, dest_dir):
|
||||||
|
@@ -124,16 +124,16 @@ def exec_command(*args, **kwargs):
|
|||||||
if isinstance(kwargs[s], AsyncPipeBase):
|
if isinstance(kwargs[s], AsyncPipeBase):
|
||||||
result[s[3:]] = kwargs[s].get_buffer()
|
result[s[3:]] = kwargs[s].get_buffer()
|
||||||
|
|
||||||
for k, v in result.items():
|
for key, value in result.items():
|
||||||
if isinstance(result[k], bytes):
|
if isinstance(value, bytes):
|
||||||
try:
|
try:
|
||||||
result[k] = result[k].decode(
|
result[key] = value.decode(
|
||||||
get_locale_encoding() or get_filesystem_encoding()
|
get_locale_encoding() or get_filesystem_encoding()
|
||||||
)
|
)
|
||||||
except UnicodeDecodeError:
|
except UnicodeDecodeError:
|
||||||
result[k] = result[k].decode("latin-1")
|
result[key] = value.decode("latin-1")
|
||||||
if v and isinstance(v, string_types):
|
if value and isinstance(value, string_types):
|
||||||
result[k] = result[k].strip()
|
result[key] = value.strip()
|
||||||
|
|
||||||
return result
|
return result
|
||||||
|
|
||||||
|
Reference in New Issue
Block a user