From b006f5301094f91595634bbb929b76f7b39a56d8 Mon Sep 17 00:00:00 2001 From: Ivan Kravets Date: Sat, 2 Jul 2022 19:03:25 +0300 Subject: [PATCH] PyLint: Fix "useless-object-inheritance" --- .pylintrc | 8 +------- platformio/app.py | 2 +- platformio/builder/tools/pioino.py | 2 +- platformio/builder/tools/piolib.py | 2 +- platformio/cache.py | 2 +- platformio/check/defect.py | 2 +- platformio/check/tools/__init__.py | 2 +- platformio/check/tools/base.py | 2 +- platformio/debug/config/factory.py | 2 +- platformio/device/list/util.py | 2 +- platformio/fs.py | 2 +- platformio/home/rpc/handlers/piocore.py | 2 +- platformio/http.py | 4 ++-- platformio/maintenance.py | 2 +- platformio/package/download.py | 2 +- platformio/package/lockfile.py | 2 +- platformio/package/manager/_download.py | 2 +- platformio/package/manager/_install.py | 2 +- platformio/package/manager/_legacy.py | 2 +- platformio/package/manager/_registry.py | 2 +- platformio/package/manager/_symlink.py | 2 +- platformio/package/manager/_uninstall.py | 2 +- platformio/package/manager/_update.py | 2 +- platformio/package/manifest/parser.py | 6 +++--- platformio/package/manifest/schema.py | 6 ++++-- platformio/package/meta.py | 10 +++++----- platformio/package/pack.py | 2 +- platformio/package/unpack.py | 4 ++-- platformio/package/vcsclient.py | 12 ++++++++---- platformio/platform/_packages.py | 2 +- platformio/platform/_run.py | 2 +- platformio/platform/board.py | 2 +- platformio/platform/factory.py | 2 +- platformio/proc.py | 2 +- platformio/project/config.py | 8 +++++--- platformio/project/generator.py | 2 +- platformio/project/options.py | 2 +- platformio/registry/mirror.py | 2 +- platformio/remote/ac/base.py | 2 +- platformio/remote/projectsync.py | 2 +- platformio/run/processor.py | 2 +- platformio/telemetry.py | 4 ++-- platformio/test/runners/factory.py | 2 +- platformio/util.py | 6 +++--- 44 files changed, 69 insertions(+), 67 deletions(-) diff --git a/.pylintrc b/.pylintrc index 8f80370c..c2f85aa2 100644 --- a/.pylintrc +++ b/.pylintrc @@ -3,16 +3,10 @@ output-format=colorized [MESSAGES CONTROL] disable= - bad-continuation, - bad-whitespace, missing-docstring, - ungrouped-imports, invalid-name, - cyclic-import, - duplicate-code, - superfluous-parens, too-few-public-methods, - useless-object-inheritance, + superfluous-parens, useless-import-alias, bad-option-value, consider-using-dict-items, diff --git a/platformio/app.py b/platformio/app.py index c9af8190..b2c78bc1 100644 --- a/platformio/app.py +++ b/platformio/app.py @@ -70,7 +70,7 @@ SESSION_VARS = { } -class State(object): +class State: def __init__(self, path=None, lock=False): self.path = path self.lock = lock diff --git a/platformio/builder/tools/pioino.py b/platformio/builder/tools/pioino.py index 0c1f59a2..453d1f51 100644 --- a/platformio/builder/tools/pioino.py +++ b/platformio/builder/tools/pioino.py @@ -26,7 +26,7 @@ import click from platformio.compat import get_filesystem_encoding, get_locale_encoding -class InoToCPPConverter(object): +class InoToCPPConverter: PROTOTYPE_RE = re.compile( r"""^( diff --git a/platformio/builder/tools/piolib.py b/platformio/builder/tools/piolib.py index 137692b8..ea16b3b3 100644 --- a/platformio/builder/tools/piolib.py +++ b/platformio/builder/tools/piolib.py @@ -46,7 +46,7 @@ from platformio.package.meta import PackageItem from platformio.project.options import ProjectOptions -class LibBuilderFactory(object): +class LibBuilderFactory: @staticmethod def new(env, path, verbose=int(ARGUMENTS.get("PIOVERBOSE", 0))): clsname = "UnknownLibBuilder" diff --git a/platformio/cache.py b/platformio/cache.py index e8b7982d..20901545 100644 --- a/platformio/cache.py +++ b/platformio/cache.py @@ -23,7 +23,7 @@ from platformio.package.lockfile import LockFile from platformio.project.helpers import get_project_cache_dir -class ContentCache(object): +class ContentCache: def __init__(self, namespace=None): self.cache_dir = os.path.join(get_project_cache_dir(), namespace or "content") self._db_path = os.path.join(self.cache_dir, "db.data") diff --git a/platformio/check/defect.py b/platformio/check/defect.py index d271ab48..15f9df70 100644 --- a/platformio/check/defect.py +++ b/platformio/check/defect.py @@ -22,7 +22,7 @@ from platformio.project.helpers import get_project_dir # pylint: disable=too-many-arguments -class DefectItem(object): +class DefectItem: SEVERITY_HIGH = 1 SEVERITY_MEDIUM = 2 diff --git a/platformio/check/tools/__init__.py b/platformio/check/tools/__init__.py index 824df5bb..58a9263d 100644 --- a/platformio/check/tools/__init__.py +++ b/platformio/check/tools/__init__.py @@ -18,7 +18,7 @@ from platformio.check.tools.cppcheck import CppcheckCheckTool from platformio.check.tools.pvsstudio import PvsStudioCheckTool -class CheckToolFactory(object): +class CheckToolFactory: @staticmethod def new(tool, project_dir, config, envname, options): cls = None diff --git a/platformio/check/tools/base.py b/platformio/check/tools/base.py index 90c58bc1..d51b3158 100644 --- a/platformio/check/tools/base.py +++ b/platformio/check/tools/base.py @@ -25,7 +25,7 @@ from platformio.package.meta import PackageSpec from platformio.project.helpers import load_build_metadata -class CheckToolBase(object): # pylint: disable=too-many-instance-attributes +class CheckToolBase: # pylint: disable=too-many-instance-attributes def __init__(self, project_dir, config, envname, options): self.config = config self.envname = envname diff --git a/platformio/debug/config/factory.py b/platformio/debug/config/factory.py index 4741b800..d7358b92 100644 --- a/platformio/debug/config/factory.py +++ b/platformio/debug/config/factory.py @@ -19,7 +19,7 @@ from platformio.debug.config.generic import GenericDebugConfig from platformio.debug.config.native import NativeDebugConfig -class DebugConfigFactory(object): +class DebugConfigFactory: @staticmethod def get_clsname(name): name = re.sub(r"[^\da-z\_\-]+", "", name, flags=re.I) diff --git a/platformio/device/list/util.py b/platformio/device/list/util.py index e82dc601..2c1d0385 100644 --- a/platformio/device/list/util.py +++ b/platformio/device/list/util.py @@ -84,7 +84,7 @@ def list_logical_devices(): def list_mdns_services(): - class mDNSListener(object): + class mDNSListener: def __init__(self): self._zc = zeroconf.Zeroconf(interfaces=zeroconf.InterfaceChoice.All) self._found_types = [] diff --git a/platformio/fs.py b/platformio/fs.py index b6678cc7..60111694 100644 --- a/platformio/fs.py +++ b/platformio/fs.py @@ -28,7 +28,7 @@ from platformio import exception, proc from platformio.compat import IS_WINDOWS -class cd(object): +class cd: def __init__(self, new_path): self.new_path = new_path self.prev_path = os.getcwd() diff --git a/platformio/home/rpc/handlers/piocore.py b/platformio/home/rpc/handlers/piocore.py index a1098f3a..7a0052d4 100644 --- a/platformio/home/rpc/handlers/piocore.py +++ b/platformio/home/rpc/handlers/piocore.py @@ -29,7 +29,7 @@ from platformio.compat import get_locale_encoding, is_bytes from platformio.home import helpers -class MultiThreadingStdStream(object): +class MultiThreadingStdStream: def __init__(self, parent_stream): self._buffers = {threading.get_ident(): parent_stream} diff --git a/platformio/http.py b/platformio/http.py index 2b31411f..ff88a85b 100644 --- a/platformio/http.py +++ b/platformio/http.py @@ -57,7 +57,7 @@ class EndpointSession(requests.Session): return super().request(method, urljoin(self.base_url, url), *args, **kwargs) -class EndpointSessionIterator(object): +class EndpointSessionIterator: def __init__(self, endpoints): if not isinstance(endpoints, list): endpoints = [endpoints] @@ -82,7 +82,7 @@ class EndpointSessionIterator(object): return session -class HTTPClient(object): +class HTTPClient: def __init__(self, endpoints): self._session_iter = EndpointSessionIterator(endpoints) self._session = None diff --git a/platformio/maintenance.py b/platformio/maintenance.py index 925d2f3a..bd9ccd9f 100644 --- a/platformio/maintenance.py +++ b/platformio/maintenance.py @@ -79,7 +79,7 @@ def set_caller(caller=None): return app.set_session_var("caller_id", caller) -class Upgrader(object): +class Upgrader: def __init__(self, from_version, to_version): self.from_version = pepver_to_semver(from_version) self.to_version = pepver_to_semver(to_version) diff --git a/platformio/package/download.py b/platformio/package/download.py index ffc57d50..f8b67ba8 100644 --- a/platformio/package/download.py +++ b/platformio/package/download.py @@ -25,7 +25,7 @@ from platformio import __default_requests_timeout__, app, fs from platformio.package.exception import PackageException -class FileDownloader(object): +class FileDownloader: def __init__(self, url, dest_dir=None): self._request = None # make connection diff --git a/platformio/package/lockfile.py b/platformio/package/lockfile.py index 3a410a60..3c6b2047 100644 --- a/platformio/package/lockfile.py +++ b/platformio/package/lockfile.py @@ -44,7 +44,7 @@ class LockFileTimeoutError(PlatformioException): pass -class LockFile(object): +class LockFile: def __init__(self, path, timeout=LOCKFILE_TIMEOUT, delay=LOCKFILE_DELAY): self.timeout = timeout self.delay = delay diff --git a/platformio/package/manager/_download.py b/platformio/package/manager/_download.py index 0c073e11..9d9c6118 100644 --- a/platformio/package/manager/_download.py +++ b/platformio/package/manager/_download.py @@ -25,7 +25,7 @@ from platformio.package.download import FileDownloader from platformio.package.lockfile import LockFile -class PackageManagerDownloadMixin(object): +class PackageManagerDownloadMixin: DOWNLOAD_CACHE_EXPIRE = 86400 * 30 # keep package in a local cache for 1 month diff --git a/platformio/package/manager/_install.py b/platformio/package/manager/_install.py index a885b68f..ae442bd0 100644 --- a/platformio/package/manager/_install.py +++ b/platformio/package/manager/_install.py @@ -26,7 +26,7 @@ from platformio.package.unpack import FileUnpacker from platformio.package.vcsclient import VCSClientFactory -class PackageManagerInstallMixin(object): +class PackageManagerInstallMixin: _INSTALL_HISTORY = None # avoid circle dependencies diff --git a/platformio/package/manager/_legacy.py b/platformio/package/manager/_legacy.py index 978efc9c..4b02a1df 100644 --- a/platformio/package/manager/_legacy.py +++ b/platformio/package/manager/_legacy.py @@ -18,7 +18,7 @@ from platformio import fs from platformio.package.meta import PackageItem, PackageSpec -class PackageManagerLegacyMixin(object): +class PackageManagerLegacyMixin: def build_legacy_spec(self, pkg_dir): # find src manifest src_manifest_name = ".piopkgmanager.json" diff --git a/platformio/package/manager/_registry.py b/platformio/package/manager/_registry.py index f861ddf4..4015cffb 100644 --- a/platformio/package/manager/_registry.py +++ b/platformio/package/manager/_registry.py @@ -23,7 +23,7 @@ from platformio.registry.client import RegistryClient from platformio.registry.mirror import RegistryFileMirrorIterator -class PackageManagerRegistryMixin(object): +class PackageManagerRegistryMixin: def install_from_registry(self, spec, search_qualifiers=None): if spec.owner and spec.name and not search_qualifiers: package = self.fetch_registry_package(spec) diff --git a/platformio/package/manager/_symlink.py b/platformio/package/manager/_symlink.py index 8c5eae38..33ff03f1 100644 --- a/platformio/package/manager/_symlink.py +++ b/platformio/package/manager/_symlink.py @@ -20,7 +20,7 @@ from platformio.package.exception import PackageException from platformio.package.meta import PackageItem, PackageSpec -class PackageManagerSymlinkMixin(object): +class PackageManagerSymlinkMixin: @staticmethod def is_symlink(path): return path and path.endswith(".pio-link") and os.path.isfile(path) diff --git a/platformio/package/manager/_uninstall.py b/platformio/package/manager/_uninstall.py index 9c6b5772..c845145a 100644 --- a/platformio/package/manager/_uninstall.py +++ b/platformio/package/manager/_uninstall.py @@ -22,7 +22,7 @@ from platformio.package.exception import UnknownPackageError from platformio.package.meta import PackageItem, PackageSpec -class PackageManagerUninstallMixin(object): +class PackageManagerUninstallMixin: def uninstall(self, spec, skip_dependencies=False): try: self.lock() diff --git a/platformio/package/manager/_update.py b/platformio/package/manager/_update.py index 5d689ba6..353e9e45 100644 --- a/platformio/package/manager/_update.py +++ b/platformio/package/manager/_update.py @@ -21,7 +21,7 @@ from platformio.package.meta import PackageItem, PackageOutdatedResult, PackageS from platformio.package.vcsclient import VCSBaseException, VCSClientFactory -class PackageManagerUpdateMixin(object): +class PackageManagerUpdateMixin: def outdated(self, pkg, spec=None): assert isinstance(pkg, PackageItem) assert pkg.metadata diff --git a/platformio/package/manifest/parser.py b/platformio/package/manifest/parser.py index 8b0c4e87..a70af7ea 100644 --- a/platformio/package/manifest/parser.py +++ b/platformio/package/manifest/parser.py @@ -27,7 +27,7 @@ from platformio.package.exception import ManifestParserError, UnknownManifestErr from platformio.project.helpers import is_platformio_project -class ManifestFileType(object): +class ManifestFileType: PLATFORM_JSON = "platform.json" LIBRARY_JSON = "library.json" LIBRARY_PROPERTIES = "library.properties" @@ -53,7 +53,7 @@ class ManifestFileType(object): return None -class ManifestParserFactory(object): +class ManifestParserFactory: @staticmethod def read_manifest_contents(path): last_err = None @@ -139,7 +139,7 @@ class ManifestParserFactory(object): raise UnknownManifestError("Unknown manifest file type %s" % type) -class BaseManifestParser(object): +class BaseManifestParser: def __init__(self, contents, remote_url=None, package_dir=None): self.remote_url = remote_url self.package_dir = package_dir diff --git a/platformio/package/manifest/schema.py b/platformio/package/manifest/schema.py index 8f8a7d16..d9eeba14 100644 --- a/platformio/package/manifest/schema.py +++ b/platformio/package/manifest/schema.py @@ -28,7 +28,7 @@ from platformio.util import memoized class BaseSchema(Schema): - class Meta(object): # pylint: disable=no-init + class Meta: # pylint: disable=no-init unknown = marshmallow.EXCLUDE # pylint: disable=no-member def load_manifest(self, data): @@ -253,7 +253,9 @@ class ManifestSchema(BaseSchema): try: spdx = self.load_spdx_licenses() except requests.exceptions.RequestException as exc: - raise ValidationError("Could not load SPDX licenses for validation") from exc + raise ValidationError( + "Could not load SPDX licenses for validation" + ) from exc known_ids = set(item.get("licenseId") for item in spdx.get("licenses", [])) if value in known_ids: return True diff --git a/platformio/package/meta.py b/platformio/package/meta.py index 20199527..34475a9c 100644 --- a/platformio/package/meta.py +++ b/platformio/package/meta.py @@ -27,7 +27,7 @@ from platformio.package.manifest.parser import ManifestFileType from platformio.package.version import cast_version_to_semver -class PackageType(object): +class PackageType: LIBRARY = "library" PLATFORM = "platform" TOOL = "tool" @@ -63,7 +63,7 @@ class PackageType(object): return None -class PackageOutdatedResult(object): +class PackageOutdatedResult: UPDATE_INCREMENT_MAJOR = "major" UPDATE_INCREMENT_MINOR = "minor" UPDATE_INCREMENT_PATCH = "patch" @@ -122,7 +122,7 @@ class PackageOutdatedResult(object): return True -class PackageSpec(object): # pylint: disable=too-many-instance-attributes +class PackageSpec: # pylint: disable=too-many-instance-attributes def __init__( # pylint: disable=redefined-builtin,too-many-arguments self, raw=None, owner=None, id=None, name=None, requirements=None, uri=None ): @@ -358,7 +358,7 @@ class PackageSpec(object): # pylint: disable=too-many-instance-attributes return name -class PackageMetaData(object): +class PackageMetaData: def __init__( # pylint: disable=redefined-builtin self, type, name, version, spec=None ): @@ -426,7 +426,7 @@ class PackageMetaData(object): return PackageMetaData(**data) -class PackageItem(object): +class PackageItem: METAFILE_NAME = ".piopm" diff --git a/platformio/package/pack.py b/platformio/package/pack.py index ed9a9a21..0170ee3d 100644 --- a/platformio/package/pack.py +++ b/platformio/package/pack.py @@ -32,7 +32,7 @@ from platformio.package.meta import PackageItem from platformio.package.unpack import FileUnpacker -class PackagePacker(object): +class PackagePacker: INCLUDE_DEFAULT = list(ManifestFileType.items().values()) + [ "README", "README.md", diff --git a/platformio/package/unpack.py b/platformio/package/unpack.py index d8544a25..e39222c2 100644 --- a/platformio/package/unpack.py +++ b/platformio/package/unpack.py @@ -31,7 +31,7 @@ class ExtractArchiveItemError(PackageException): ) -class BaseArchiver(object): +class BaseArchiver: def __init__(self, arhfileobj): self._afo = arhfileobj @@ -129,7 +129,7 @@ class ZIPArchiver(BaseArchiver): self.preserve_mtime(item, dest_dir) -class FileUnpacker(object): +class FileUnpacker: def __init__(self, path): self.path = path self._archiver = None diff --git a/platformio/package/vcsclient.py b/platformio/package/vcsclient.py index 9299ada9..b4dafc03 100644 --- a/platformio/package/vcsclient.py +++ b/platformio/package/vcsclient.py @@ -29,7 +29,7 @@ class VCSBaseException(PackageException): pass -class VCSClientFactory(object): +class VCSClientFactory: @staticmethod def new(src_dir, remote_url, silent=False): result = urlparse(remote_url) @@ -52,10 +52,12 @@ class VCSClientFactory(object): assert isinstance(obj, VCSClientBase) return obj except (KeyError, AssertionError) as exc: - raise VCSBaseException("VCS: Unknown repository type %s" % remote_url) from exc + raise VCSBaseException( + "VCS: Unknown repository type %s" % remote_url + ) from exc -class VCSClientBase(object): +class VCSClientBase: command = None @@ -109,7 +111,9 @@ class VCSClientBase(object): subprocess.check_call(args, **kwargs) return True except subprocess.CalledProcessError as exc: - raise VCSBaseException("VCS: Could not process command %s" % exc.cmd) from exc + raise VCSBaseException( + "VCS: Could not process command %s" % exc.cmd + ) from exc def get_cmd_output(self, args, **kwargs): args = [self.command] + args diff --git a/platformio/platform/_packages.py b/platformio/platform/_packages.py index c741c790..2a3aa490 100644 --- a/platformio/platform/_packages.py +++ b/platformio/platform/_packages.py @@ -15,7 +15,7 @@ from platformio.package.meta import PackageSpec -class PlatformPackagesMixin(object): +class PlatformPackagesMixin: def get_package_spec(self, name, version=None): return PackageSpec( owner=self.packages[name].get("owner"), diff --git a/platformio/platform/_run.py b/platformio/platform/_run.py index 5bac13ae..79afff94 100644 --- a/platformio/platform/_run.py +++ b/platformio/platform/_run.py @@ -27,7 +27,7 @@ from platformio.package.manager.core import get_core_package_dir from platformio.platform.exception import BuildScriptNotFound -class PlatformRunMixin(object): +class PlatformRunMixin: LINE_ERROR_RE = re.compile(r"(^|\s+)error:?\s+", re.I) diff --git a/platformio/platform/board.py b/platformio/platform/board.py index fa0cdf9c..73a3ebd7 100644 --- a/platformio/platform/board.py +++ b/platformio/platform/board.py @@ -21,7 +21,7 @@ from platformio.exception import UserSideException from platformio.platform.exception import InvalidBoardManifest -class PlatformBoardConfig(object): +class PlatformBoardConfig: def __init__(self, manifest_path): self._id = os.path.basename(manifest_path)[:-5] assert os.path.isfile(manifest_path) diff --git a/platformio/platform/factory.py b/platformio/platform/factory.py index 72b44790..b628590c 100644 --- a/platformio/platform/factory.py +++ b/platformio/platform/factory.py @@ -23,7 +23,7 @@ from platformio.platform import base from platformio.platform.exception import UnknownPlatform -class PlatformFactory(object): +class PlatformFactory: @staticmethod def get_clsname(name): name = re.sub(r"[^\da-z\_]+", "", name, flags=re.I) diff --git a/platformio/proc.py b/platformio/proc.py index f32fe715..fb66bc7f 100644 --- a/platformio/proc.py +++ b/platformio/proc.py @@ -27,7 +27,7 @@ from platformio.compat import ( ) -class AsyncPipeBase(object): +class AsyncPipeBase: def __init__(self): self._fd_read, self._fd_write = os.pipe() self._pipe_reader = os.fdopen( diff --git a/platformio/project/config.py b/platformio/project/config.py index e9440bc9..3443863a 100644 --- a/platformio/project/config.py +++ b/platformio/project/config.py @@ -38,7 +38,7 @@ CONFIG_HEADER = """ """ -class ProjectConfigBase(object): +class ProjectConfigBase: INLINE_COMMENT_RE = re.compile(r"\s+;.*$") VARTPL_RE = re.compile(r"\$\{([^\.\}\()]+)\.([^\}]+)\}") @@ -352,7 +352,9 @@ class ProjectConfigBase(object): except click.BadParameter as exc: if not self.expand_interpolations: return value - raise exception.ProjectOptionValueError(exc.format_message(), option, section) + raise exception.ProjectOptionValueError( + exc.format_message(), option, section + ) @staticmethod def cast_to(value, to_type): @@ -394,7 +396,7 @@ class ProjectConfigBase(object): return True -class ProjectConfigDirsMixin(object): +class ProjectConfigDirsMixin: def get_optional_dir(self, name): """ Deprecated, used by platformio-node-helpers.project.observer.fetchLibDirs diff --git a/platformio/project/generator.py b/platformio/project/generator.py index 10efba7d..cca74dbf 100644 --- a/platformio/project/generator.py +++ b/platformio/project/generator.py @@ -23,7 +23,7 @@ from platformio.proc import where_is_program from platformio.project.helpers import load_build_metadata -class ProjectGenerator(object): +class ProjectGenerator: def __init__(self, config, env_name, ide, board_ids=None): self.config = config self.project_dir = os.path.dirname(config.path) diff --git a/platformio/project/options.py b/platformio/project/options.py index 21a9a130..4c29b925 100644 --- a/platformio/project/options.py +++ b/platformio/project/options.py @@ -24,7 +24,7 @@ from platformio import fs from platformio.compat import IS_WINDOWS, hashlib_encode_data -class ConfigOption(object): # pylint: disable=too-many-instance-attributes +class ConfigOption: # pylint: disable=too-many-instance-attributes def __init__( self, scope, diff --git a/platformio/registry/mirror.py b/platformio/registry/mirror.py index d967838e..4235e88d 100644 --- a/platformio/registry/mirror.py +++ b/platformio/registry/mirror.py @@ -21,7 +21,7 @@ from platformio.http import HTTPClient from platformio.registry.client import RegistryClient -class RegistryFileMirrorIterator(object): +class RegistryFileMirrorIterator: HTTP_CLIENT_INSTANCES = {} diff --git a/platformio/remote/ac/base.py b/platformio/remote/ac/base.py index e92118c9..8105062c 100644 --- a/platformio/remote/ac/base.py +++ b/platformio/remote/ac/base.py @@ -16,7 +16,7 @@ from twisted.internet import defer # pylint: disable=import-error from twisted.spread import pb # pylint: disable=import-error -class AsyncCommandBase(object): +class AsyncCommandBase: MAX_BUFFER_SIZE = 1024 * 1024 # 1Mb diff --git a/platformio/remote/projectsync.py b/platformio/remote/projectsync.py index 867922bd..820034ae 100644 --- a/platformio/remote/projectsync.py +++ b/platformio/remote/projectsync.py @@ -31,7 +31,7 @@ class PROJECT_SYNC_STAGE(constants.Flags): COMPLETED = constants.FlagConstant() -class ProjectSync(object): +class ProjectSync: def __init__(self, path): self.path = path if not isdir(self.path): diff --git a/platformio/run/processor.py b/platformio/run/processor.py index 4751d352..00a96f02 100644 --- a/platformio/run/processor.py +++ b/platformio/run/processor.py @@ -20,7 +20,7 @@ from platformio.test.runners.base import CTX_META_TEST_RUNNING_NAME # pylint: disable=too-many-instance-attributes -class EnvironmentProcessor(object): +class EnvironmentProcessor: def __init__( # pylint: disable=too-many-arguments self, cmd_ctx, diff --git a/platformio/telemetry.py b/platformio/telemetry.py index 8606e48e..a3aded5a 100644 --- a/platformio/telemetry.py +++ b/platformio/telemetry.py @@ -34,7 +34,7 @@ from platformio.proc import is_ci, is_container from platformio.project.helpers import is_platformio_project -class TelemetryBase(object): +class TelemetryBase: def __init__(self): self._params = {} @@ -198,7 +198,7 @@ class MeasurementProtocol(TelemetryBase): @util.singleton -class MPDataPusher(object): +class MPDataPusher: MAX_WORKERS = 5 diff --git a/platformio/test/runners/factory.py b/platformio/test/runners/factory.py index 64381807..ecb6f884 100644 --- a/platformio/test/runners/factory.py +++ b/platformio/test/runners/factory.py @@ -23,7 +23,7 @@ from platformio.test.result import TestSuite from platformio.test.runners.base import TestRunnerBase, TestRunnerOptions -class TestRunnerFactory(object): +class TestRunnerFactory: @staticmethod def get_clsname(name): name = re.sub(r"[^\da-z\_\-]+", "", name, flags=re.I) diff --git a/platformio/util.py b/platformio/util.py index 45af8b12..5a8b1923 100644 --- a/platformio/util.py +++ b/platformio/util.py @@ -38,7 +38,7 @@ from platformio.proc import exec_command get_serialports = get_serial_ports -class memoized(object): +class memoized: def __init__(self, expire=0): expire = str(expire) if expire.isdigit(): @@ -65,7 +65,7 @@ class memoized(object): self.cache.clear() -class throttle(object): +class throttle: def __init__(self, threshhold): self.threshhold = threshhold # milliseconds self.last = 0 @@ -97,7 +97,7 @@ class RetryStopException(RetryException): pass -class retry(object): +class retry: RetryNextException = RetryNextException RetryStopException = RetryStopException