From b94457217280145eaabbf308d3745e5deade18ce Mon Sep 17 00:00:00 2001 From: Lucjan Dudek Date: Wed, 1 Feb 2023 11:59:24 +0100 Subject: [PATCH] PR minor fixes --- goth/api_monitor/api_events.py | 4 +--- goth/runner/probe/__init__.py | 2 +- pyproject.toml | 18 +++++++++--------- 3 files changed, 11 insertions(+), 13 deletions(-) diff --git a/goth/api_monitor/api_events.py b/goth/api_monitor/api_events.py index fec5e5b6..9aaa3e94 100644 --- a/goth/api_monitor/api_events.py +++ b/goth/api_monitor/api_events.py @@ -107,9 +107,7 @@ def content(self) -> str: return self.http_response.content.decode("utf-8") def __str__(self) -> str: - return ( - f"[response ({self.status_code})] {self.request.header_str}; body: {self.content}" - ) + return f"[response ({self.status_code})] {self.request.header_str}; body: {self.content}" class APIError(APIEvent): diff --git a/goth/runner/probe/__init__.py b/goth/runner/probe/__init__.py index b43369d5..8cf6771f 100644 --- a/goth/runner/probe/__init__.py +++ b/goth/runner/probe/__init__.py @@ -351,7 +351,7 @@ async def run_command_on_host( command: str, env: Optional[Mapping[str, str]] = None, command_timeout: float = 300, - ) -> AsyncIterator[Tuple[asyncio.Task, PatternMatchingEventMonitor, process.ProcessMonitor],]: + ) -> AsyncIterator[Tuple[asyncio.Task, PatternMatchingEventMonitor, process.ProcessMonitor]]: """Run `command` on host in given `env` and with optional `timeout`. The command is run in the environment extending `env` with variables needed diff --git a/pyproject.toml b/pyproject.toml index 2c741077..8c801866 100644 --- a/pyproject.toml +++ b/pyproject.toml @@ -76,16 +76,16 @@ max-line-length = 100 extend-ignore = [ "E203", # See https://github.com/PyCQA/pycodestyle/issues/373 "E231", # black ignores this rule when formatting - "D100", # No docs for public module https://github.com/golemfactory/goth/issues/606 - "D101", # No docs for public class https://github.com/golemfactory/goth/issues/606 - "D102", # No docs for public method https://github.com/golemfactory/goth/issues/606 - "D104", # No docs for public package https://github.com/golemfactory/goth/issues/606 - "D105", # No docs for magic method https://github.com/golemfactory/goth/issues/606 - "D107", # No docs for __init__ https://github.com/golemfactory/goth/issues/606 + "D100", # No docs for public module: to be fixed by https://github.com/golemfactory/goth/issues/606 + "D101", # No docs for public class: to be fixed by https://github.com/golemfactory/goth/issues/606 + "D102", # No docs for public method: to be fixed by https://github.com/golemfactory/goth/issues/606 + "D104", # No docs for public package: to be fixed by https://github.com/golemfactory/goth/issues/606 + "D105", # No docs for magic method: to be fixed by https://github.com/golemfactory/goth/issues/606 + "D107", # No docs for __init__: to be fixed by https://github.com/golemfactory/goth/issues/606 "D202", # We prefer whitelines after docstrings - "D400", # First line should end with a period https://github.com/golemfactory/goth/issues/606 - "D403", # First word of the first line should be properly capitalize https://github.com/golemfactory/goth/issues/606 - "F405", # Star imports https://github.com/golemfactory/goth/issues/607 + "D400", # First line should end with a period: to be fixed by https://github.com/golemfactory/goth/issues/606 + "D403", # First word of the first line should be properly capitalize: to be fixed by https://github.com/golemfactory/goth/issues/606 + "F405", # Star imports: to be fixed by https://github.com/golemfactory/goth/issues/607 ] exclude = [