From d10baeb2364315f9b2d78c146ed26c71cc683dd7 Mon Sep 17 00:00:00 2001 From: Michael Graeb Date: Mon, 2 Oct 2023 22:19:46 +0000 Subject: [PATCH] fix mypy warnings --- runners/s3-benchrunner-cli/benchrunner.py | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/runners/s3-benchrunner-cli/benchrunner.py b/runners/s3-benchrunner-cli/benchrunner.py index fd539a8f..da24e6e8 100755 --- a/runners/s3-benchrunner-cli/benchrunner.py +++ b/runners/s3-benchrunner-cli/benchrunner.py @@ -1,6 +1,6 @@ #!/usr/bin/env python3 import argparse -import boto3 +import boto3 # type: ignore from dataclasses import dataclass import json import os @@ -10,7 +10,7 @@ import sys import tempfile import time -from typing import Optional +from typing import Optional, Tuple PARSER = argparse.ArgumentParser( description='Benchmark runner for AWS CLI') @@ -127,7 +127,7 @@ def _derive_cli_config(self) -> str: ''] # blank line at end of file return '\n'.join(lines) - def _derive_cli_cmd(self) -> (list[str], Optional[str]): + def _derive_cli_cmd(self) -> Tuple[list[str], Optional[bytes]]: """ Figures out single CLI command that will do everything in the benchmark. Exits with skip code if we can't do this benchmark in one CLI command. @@ -138,7 +138,7 @@ def _derive_cli_cmd(self) -> (list[str], Optional[str]): first_task = self.config.tasks[0] cmd = ['aws', 's3', 'cp'] - stdin = None + stdin: Optional[bytes] = None if num_tasks == 1: # doing 1 file is simple, just name the src and dst