chore: fix broken pre-commit hooks (#60)

* chore: bump pre-commit hook versions for Python 3.12 / click 8.1

black 20.8b1 was broken against click>=8.1 (ImportError: _unicodefun),
and flake8 3.9.2 was broken on Python 3.12 (EntryPoints.get removed).
Bumps black, flake8, their transitive plugins, and blacken-docs to
currently-maintained versions.

* chore: remove invalid 'safe = true' from [tool.black] in pyproject.toml

Modern black does not recognize 'safe' as a pyproject config key and
emits "Invalid config keys detected: 'safe'" on every run. --safe is
still the default CLI behavior, so removing the key is a no-op for
actual formatting.

* chore: apply black 24 reformat and fix flake8 lint errors

Applies black 24.10.0 formatting to the repo (mostly trailing-comma
and blank-line-after-module-docstring changes from 4 years of black
style evolution) and fixes the 29 E501 line-length errors plus 1 D419
empty-docstring that the upgraded flake8 surfaced. Bundled together
because the formatting hook and the lint hook block each other in
intermediate states — keeping them in one commit avoids a red
midpoint in the series.

* chore: bump black and flake8 dev-deps to align with pre-commit

black ^22.0.0 -> ^24.0.0 and flake8 ^6.0.0 -> ^7.0.0 so that
'poetry run black' / 'poetry run flake8' (and editor integrations
using the poetry venv) produce the same output as the pre-commit
hooks. Regenerates poetry.lock; transitive bumps are scoped to
black/flake8 deps only (packaging, pycodestyle, pyflakes,
typing-extensions).

* chore: pin pre-commit flake8 to 7.3.0 to match poetry-resolved version

Keeps the pre-commit hook and the poetry dev-dep on the exact same
flake8 version so local and CI runs can't diverge.

Author: micaelaw

.pre-commit-config.yaml

@@ -1,12 +1,12 @@
 repos:
   - repo: https://github.com/psf/black
-    rev: 20.8b1
+    rev: 24.10.0
     hooks:
       - id: black
         args: [ "--safe", "--diff", "--color"]
         verbose: true
   - repo: https://github.com/psf/black
-    rev: 20.8b1
+    rev: 24.10.0
     hooks:
       - id: black
         args: [ "--safe", "--quiet"]
@@ -17,10 +17,10 @@ repos:
       - id: isort
         args: ["--profile", "black", "--filter-files"]
   - repo: https://github.com/asottile/blacken-docs
-    rev: v1.10.0
+    rev: 1.19.1
     hooks:
       - id: blacken-docs
-        additional_dependencies: [ black==20.8b1 ]
+        additional_dependencies: [ black==24.10.0 ]
   - repo: https://github.com/pre-commit/pre-commit-hooks
     rev: v4.0.1
     hooks:
@@ -41,10 +41,10 @@ repos:
       - id: python-use-type-annotations
       - id: rst-backticks
   - repo: https://github.com/pycqa/flake8
-    rev: 3.9.2
+    rev: 7.3.0
     hooks:
       - id: flake8
         language_version: python3
         additional_dependencies:
-          - flake8-typing-imports==1.9.0
-          - flake8-docstrings==1.5.0
+          - flake8-typing-imports==1.15.0
+          - flake8-docstrings==1.7.0

docs/list_labels_by_tracking_number.md

@@ -31,7 +31,9 @@ def list_labels_by_tracking_number_demo():
         {"api_key": api_key, "page_size": 75, "retries": 3, "timeout": 10}
     )
     try:
-        result = shipengine.list_labels_by_tracking_number(tracking_number="1ZXXXXXXXXXXXXXXXX")
+        result = shipengine.list_labels_by_tracking_number(
+            tracking_number="1ZXXXXXXXXXXXXXXXX"
+        )
         print("::SUCCESS::")
         print(result)
     except ShipEngineError as err:

poetry.lock

@@ -44,8 +44,8 @@ fuuid = "^0.1.0"
 pytest = "^7.0.0"
 pytest-cov = "^2.11.1"
 pytest-mock = "^3.6.1"
-black = "^22.0.0"
-flake8 = "^6.0.0"
+black = "^24.0.0"
+flake8 = "^7.0.0"
 Sphinx = "^3.5.2"
 tox = "^3.23.0"
 coverage = "^5.5"
@@ -58,7 +58,6 @@ responses = "^0.13.3"
 [tool.black]
 line-length = 100
 target-version = ["py310"]
-safe = true
 
 [tool.isort]
 profile = "black"

pyproject.toml

@@ -1,4 +1,5 @@
 """ShipEngine SDK."""
+
 __version__ = "2.0.5"
 
 import logging

shipengine/__init__.py

@@ -1,4 +1,5 @@
 """ShipEngine SDK Enumerations"""
+
 from enum import Enum
 
 from .country import Country

shipengine/enums/__init__.py

@@ -1,4 +1,5 @@
 """An enumeration of valid/supported countries."""
+
 from enum import Enum
 
 

shipengine/enums/country.py

@@ -1,4 +1,5 @@
 """Enumeration of error sources returned by ShipEngine API."""
+
 from enum import Enum
 
 

shipengine/enums/error_code.py

@@ -1,4 +1,5 @@
 """Enumeration of error sources returned by ShipEngine API."""
+
 from enum import Enum
 
 

shipengine/enums/error_source.py

@@ -1,9 +1,13 @@
 """Enumeration of error types returned by ShipEngine API."""
+
 from enum import Enum
 
 
 class ErrorType(Enum):
-    """Indicates the type of an error. Think of this as a broad category rather than a specific error."""
+    """Indicates the type of an error.
+
+    Think of this as a broad category rather than a specific error.
+    """
 
     ACCOUNT_STATUS = "account_status"
     """