Skip to content

Commit

Permalink
Merge pull request #295 from boto/pre-commit-updates
Browse files Browse the repository at this point in the history
Pre-commit updates
  • Loading branch information
nateprewitt committed Dec 7, 2023
2 parents a23c0ec + 7f24a0f commit 76c3a75
Show file tree
Hide file tree
Showing 6 changed files with 5 additions and 9 deletions.
8 changes: 4 additions & 4 deletions .pre-commit-config.yaml
Original file line number Diff line number Diff line change
@@ -1,13 +1,13 @@
exclude: ^(.changes/|CHANGELOG.rst|s3transfer/compat.py)
repos:
- repo: 'https://github.com/pre-commit/pre-commit-hooks'
rev: v4.2.0
rev: v4.5.0
hooks:
- id: check-yaml
- id: end-of-file-fixer
- id: trailing-whitespace
- repo: 'https://github.com/asottile/pyupgrade'
rev: v2.32.1
rev: v3.15.0
hooks:
- id: pyupgrade
args:
Expand All @@ -17,11 +17,11 @@ repos:
hooks:
- id: isort
- repo: 'https://github.com/psf/black'
rev: 22.3.0
rev: 23.11.0
hooks:
- id: black
- repo: 'https://github.com/pycqa/flake8'
rev: 4.0.1
rev: 6.1.0
hooks:
- id: flake8
- repo: 'https://github.com/codespell-project/codespell'
Expand Down
1 change: 0 additions & 1 deletion s3transfer/__init__.py
Original file line number Diff line number Diff line change
Expand Up @@ -681,7 +681,6 @@ def __init__(


class S3Transfer:

ALLOWED_DOWNLOAD_ARGS = [
'VersionId',
'SSECustomerAlgorithm',
Expand Down
1 change: 1 addition & 0 deletions s3transfer/futures.py
Original file line number Diff line number Diff line change
Expand Up @@ -505,6 +505,7 @@ def add_done_callback(self, fn):
than concurrent.futures.Future.add_done_callback that requires
a single argument for the future.
"""

# The done callback for concurrent.futures.Future will always pass a
# the future in as the only argument. So we need to create the
# proper signature wrapper that will invoke the callback provided.
Expand Down
2 changes: 0 additions & 2 deletions tests/functional/test_copy.py
Original file line number Diff line number Diff line change
Expand Up @@ -84,7 +84,6 @@ def add_successful_copy_responses(
expected_create_mpu_params=None,
expected_complete_mpu_params=None,
):

# Add all responses needed to do the copy of the object.
# Should account for both ranged and nonranged downloads.
stubbed_responses = self.create_stubbed_responses()[1:]
Expand Down Expand Up @@ -497,7 +496,6 @@ def _get_expected_params(self):
def _add_params_to_expected_params(
self, add_copy_kwargs, operation_types, new_params
):

expected_params_to_update = []
for operation_type in operation_types:
add_copy_kwargs_key = 'expected_' + operation_type + '_params'
Expand Down
1 change: 0 additions & 1 deletion tests/functional/test_delete.py
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,6 @@


class TestDeleteObject(BaseGeneralInterfaceTest):

__test__ = True

def setUp(self):
Expand Down
1 change: 0 additions & 1 deletion tests/unit/test_s3transfer.py
Original file line number Diff line number Diff line change
Expand Up @@ -377,7 +377,6 @@ def test_multipart_upload_is_aborted_on_error(self):


class TestMultipartDownloader(unittest.TestCase):

maxDiff = None

def test_multipart_download_uses_correct_client_calls(self):
Expand Down

0 comments on commit 76c3a75

Please sign in to comment.