Merge e80ad90 into 287c85d

stripe · May 10, 2024 · e347d8c · e347d8c
2 parents 287c85d + e80ad90
commit e347d8c
Show file tree

Hide file tree

Showing 135 changed files with 581 additions and 712 deletions.
diff --git a/.flake8 b/.flake8
@@ -4,7 +4,9 @@
 # length, but can go over in some cases.
 # W503 goes against PEP8 rules. It's disabled by default, but must be disabled
 # explicitly when using `ignore`.
-ignore = E501, W503
+# E704 is disabled in the default configuration, but by specifying `ignore`, we wipe that out.
+# ruff formatting creates code that violates it, so we have to disable it manually
+ignore = E501, W503, E704
 per-file-ignores =
     */__init__.py: IMP100, E402, F401
     # we test various import patterns

diff --git a/.git-blame-ignore-revs b/.git-blame-ignore-revs
@@ -0,0 +1,3 @@
+# .git-blame-ignore-revs
+# mass formatted w/ ruff (2024-05-10)
+c46b4b950ea77e9d18ab98885cda721b3de247c0
diff --git a/.vscode/extensions.json b/.vscode/extensions.json
@@ -3,5 +3,6 @@
     "ms-python.python",
     "EditorConfig.editorconfig",
     "ms-python.flake8",
+    "charliermarsh.ruff"
   ]
 }
diff --git a/README.md b/README.md
@@ -369,7 +369,7 @@ Run the linter with:
 make lint
 ```
 
-The library uses [Black][black] for code formatting. Code must be formatted
+The library uses [Ruff][ruff] for code formatting. Code must be formatted
 with Black before PRs are submitted, otherwise CI will fail. Run the formatter
 with:
 
@@ -378,7 +378,7 @@ make fmt
 ```
 
 [api-keys]: https://dashboard.stripe.com/account/apikeys
-[black]: https://github.com/ambv/black
+[ruff]: https://github.com/astral-sh/ruff
 [connect]: https://stripe.com/connect
 [poetry]: https://github.com/sdispater/poetry
 [stripe-mock]: https://github.com/stripe/stripe-mock

diff --git a/examples/oauth.py b/examples/oauth.py
@@ -33,9 +33,7 @@ def callback():
 <p>Success! Account <code>{stripe_user_id}</code> is connected.</p>
 <p>Click <a href="/deauthorize?stripe_user_id={stripe_user_id}">here</a> to
 disconnect the account.</p>
-""".format(
-        stripe_user_id=resp["stripe_user_id"]
-    )
+""".format(stripe_user_id=resp["stripe_user_id"])
 
 
 @app.route("/deauthorize")
@@ -49,9 +47,7 @@ def deauthorize():
     return """
 <p>Success! Account <code>{stripe_user_id}</code> is disconnected.</p>
 <p>Click <a href="/">here</a> to restart the OAuth flow.</p>
-""".format(
-        stripe_user_id=stripe_user_id
-    )
+""".format(stripe_user_id=stripe_user_id)
 
 
 if __name__ == "__main__":

diff --git a/pyproject.toml b/pyproject.toml
@@ -1,33 +1,18 @@
-[tool.black]
+[tool.ruff]
+# same as our black config
 line-length = 79
-target-version = [
-  "py35",
-  "py36",
-  "py37",
-  "py38",
-  "py39",
-  "py310",
-  #   "py311",  # black 21.12b0 doesn't
-  #   "py312",  # support these targets
-]
-exclude = '''
-/(
-    \.eggs/
-  | \.git/
-  | \.tox/
-  | \.venv/
-  | _build/
-  | build/
-  | dist/
-  | venv/
-)
-'''
+extend-exclude = ["build"]
+
+[tool.ruff.format]
+# currently the default value, but opt-out in the future
+docstring-code-format = false
+
 [tool.pyright]
 include = [
   "stripe",
   "tests/test_generated_examples.py",
   "tests/test_exports.py",
-  "tests/test_http_client.py"
+  "tests/test_http_client.py",
 ]
 exclude = ["build", "**/__pycache__"]
 reportMissingTypeArgument = true

diff --git a/requirements.txt b/requirements.txt
@@ -5,7 +5,7 @@ tox == 4.5.0
 #Virtualenv 20.22.0 dropped support for all Python versions smaller or equal to Python 3.6.
 virtualenv<20.22.0
 pyright == 1.1.336
-black == 22.8.0
+ruff == 0.4.4
 flake8
 mypy == 1.7.0
 

diff --git a/stripe/_account.py b/stripe/_account.py
@@ -4004,7 +4004,6 @@ def list(
             params=params,
         )
         if not isinstance(result, ListObject):
-
             raise TypeError(
                 "Expected list object from API, got %s"
                 % (type(result).__name__)
@@ -4025,7 +4024,6 @@ async def list_async(
             params=params,
         )
         if not isinstance(result, ListObject):
-
             raise TypeError(
                 "Expected list object from API, got %s"
                 % (type(result).__name__)
@@ -4322,7 +4320,7 @@ def retrieve_capability(
         cls,
         account: str,
         capability: str,
-        **params: Unpack["Account.RetrieveCapabilityParams"]
+        **params: Unpack["Account.RetrieveCapabilityParams"],
     ) -> "Capability":
         """
         Retrieves information about the specified Account Capability.
@@ -4344,7 +4342,7 @@ async def retrieve_capability_async(
         cls,
         account: str,
         capability: str,
-        **params: Unpack["Account.RetrieveCapabilityParams"]
+        **params: Unpack["Account.RetrieveCapabilityParams"],
     ) -> "Capability":
         """
         Retrieves information about the specified Account Capability.
@@ -4366,7 +4364,7 @@ def modify_capability(
         cls,
         account: str,
         capability: str,
-        **params: Unpack["Account.ModifyCapabilityParams"]
+        **params: Unpack["Account.ModifyCapabilityParams"],
     ) -> "Capability":
         """
         Updates an existing Account Capability. Request or remove a capability by updating its requested parameter.
@@ -4388,7 +4386,7 @@ async def modify_capability_async(
         cls,
         account: str,
         capability: str,
-        **params: Unpack["Account.ModifyCapabilityParams"]
+        **params: Unpack["Account.ModifyCapabilityParams"],
     ) -> "Capability":
         """
         Updates an existing Account Capability. Request or remove a capability by updating its requested parameter.
@@ -4445,7 +4443,7 @@ async def list_capabilities_async(
     def create_external_account(
         cls,
         account: str,
-        **params: Unpack["Account.CreateExternalAccountParams"]
+        **params: Unpack["Account.CreateExternalAccountParams"],
     ) -> Union["BankAccount", "Card"]:
         """
         Create an external account for a given account.
@@ -4465,7 +4463,7 @@ def create_external_account(
     async def create_external_account_async(
         cls,
         account: str,
-        **params: Unpack["Account.CreateExternalAccountParams"]
+        **params: Unpack["Account.CreateExternalAccountParams"],
     ) -> Union["BankAccount", "Card"]:
         """
         Create an external account for a given account.
@@ -4486,7 +4484,7 @@ def retrieve_external_account(
         cls,
         account: str,
         id: str,
-        **params: Unpack["Account.RetrieveExternalAccountParams"]
+        **params: Unpack["Account.RetrieveExternalAccountParams"],
     ) -> Union["BankAccount", "Card"]:
         """
         Retrieve a specified external account for a given account.
@@ -4507,7 +4505,7 @@ async def retrieve_external_account_async(
         cls,
         account: str,
         id: str,
-        **params: Unpack["Account.RetrieveExternalAccountParams"]
+        **params: Unpack["Account.RetrieveExternalAccountParams"],
     ) -> Union["BankAccount", "Card"]:
         """
         Retrieve a specified external account for a given account.
@@ -4528,7 +4526,7 @@ def modify_external_account(
         cls,
         account: str,
         id: str,
-        **params: Unpack["Account.ModifyExternalAccountParams"]
+        **params: Unpack["Account.ModifyExternalAccountParams"],
     ) -> Union["BankAccount", "Card"]:
         """
         Updates the metadata, account holder name, account holder type of a bank account belonging to
@@ -4556,7 +4554,7 @@ async def modify_external_account_async(
         cls,
         account: str,
         id: str,
-        **params: Unpack["Account.ModifyExternalAccountParams"]
+        **params: Unpack["Account.ModifyExternalAccountParams"],
     ) -> Union["BankAccount", "Card"]:
         """
         Updates the metadata, account holder name, account holder type of a bank account belonging to
@@ -4584,7 +4582,7 @@ def delete_external_account(
         cls,
         account: str,
         id: str,
-        **params: Unpack["Account.DeleteExternalAccountParams"]
+        **params: Unpack["Account.DeleteExternalAccountParams"],
     ) -> Union["BankAccount", "Card"]:
         """
         Delete a specified external account for a given account.
@@ -4605,7 +4603,7 @@ async def delete_external_account_async(
         cls,
         account: str,
         id: str,
-        **params: Unpack["Account.DeleteExternalAccountParams"]
+        **params: Unpack["Account.DeleteExternalAccountParams"],
     ) -> Union["BankAccount", "Card"]:
         """
         Delete a specified external account for a given account.
@@ -4625,7 +4623,7 @@ async def delete_external_account_async(
     def list_external_accounts(
         cls,
         account: str,
-        **params: Unpack["Account.ListExternalAccountsParams"]
+        **params: Unpack["Account.ListExternalAccountsParams"],
     ) -> ListObject[Union["BankAccount", "Card"]]:
         """
         List external accounts for an account.
@@ -4645,7 +4643,7 @@ def list_external_accounts(
     async def list_external_accounts_async(
         cls,
         account: str,
-        **params: Unpack["Account.ListExternalAccountsParams"]
+        **params: Unpack["Account.ListExternalAccountsParams"],
     ) -> ListObject[Union["BankAccount", "Card"]]:
         """
         List external accounts for an account.
@@ -4742,7 +4740,7 @@ def retrieve_person(
         cls,
         account: str,
         person: str,
-        **params: Unpack["Account.RetrievePersonParams"]
+        **params: Unpack["Account.RetrievePersonParams"],
     ) -> "Person":
         """
         Retrieves an existing person.
@@ -4763,7 +4761,7 @@ async def retrieve_person_async(
         cls,
         account: str,
         person: str,
-        **params: Unpack["Account.RetrievePersonParams"]
+        **params: Unpack["Account.RetrievePersonParams"],
     ) -> "Person":
         """
         Retrieves an existing person.
@@ -4784,7 +4782,7 @@ def modify_person(
         cls,
         account: str,
         person: str,
-        **params: Unpack["Account.ModifyPersonParams"]
+        **params: Unpack["Account.ModifyPersonParams"],
     ) -> "Person":
         """
         Updates an existing person.
@@ -4805,7 +4803,7 @@ async def modify_person_async(
         cls,
         account: str,
         person: str,
-        **params: Unpack["Account.ModifyPersonParams"]
+        **params: Unpack["Account.ModifyPersonParams"],
     ) -> "Person":
         """
         Updates an existing person.
@@ -4826,7 +4824,7 @@ def delete_person(
         cls,
         account: str,
         person: str,
-        **params: Unpack["Account.DeletePersonParams"]
+        **params: Unpack["Account.DeletePersonParams"],
     ) -> "Person":
         """
         Deletes an existing person's relationship to the account's legal entity. Any person with a relationship for an account can be deleted through the API, except if the person is the account_opener. If your integration is using the executive parameter, you cannot delete the only verified executive on file.
@@ -4847,7 +4845,7 @@ async def delete_person_async(
         cls,
         account: str,
         person: str,
-        **params: Unpack["Account.DeletePersonParams"]
+        **params: Unpack["Account.DeletePersonParams"],
     ) -> "Person":
         """
         Deletes an existing person's relationship to the account's legal entity. Any person with a relationship for an account can be deleted through the API, except if the person is the account_opener. If your integration is using the executive parameter, you cannot delete the only verified executive on file.

diff --git a/stripe/_api_requestor.py b/stripe/_api_requestor.py
@@ -511,9 +511,9 @@ def _args_for_request_with_retries(
                 generator = MultipartDataGenerator()
                 generator.add_params(params or {})
                 post_data = generator.get_post_data()
-                headers[
-                    "Content-Type"
-                ] = "multipart/form-data; boundary=%s" % (generator.boundary,)
+                headers["Content-Type"] = (
+                    "multipart/form-data; boundary=%s" % (generator.boundary,)
+                )
             else:
                 post_data = encoded_body
         else:

diff --git a/stripe/_apple_pay_domain.py b/stripe/_apple_pay_domain.py
@@ -217,7 +217,6 @@ def list(
             params=params,
         )
         if not isinstance(result, ListObject):
-
             raise TypeError(
                 "Expected list object from API, got %s"
                 % (type(result).__name__)
@@ -238,7 +237,6 @@ async def list_async(
             params=params,
         )
         if not isinstance(result, ListObject):
-
             raise TypeError(
                 "Expected list object from API, got %s"
                 % (type(result).__name__)

diff --git a/stripe/_application_fee.py b/stripe/_application_fee.py
@@ -209,7 +209,6 @@ def list(
             params=params,
         )
         if not isinstance(result, ListObject):
-
             raise TypeError(
                 "Expected list object from API, got %s"
                 % (type(result).__name__)
@@ -230,7 +229,6 @@ async def list_async(
             params=params,
         )
         if not isinstance(result, ListObject):
-
             raise TypeError(
                 "Expected list object from API, got %s"
                 % (type(result).__name__)
@@ -483,7 +481,7 @@ def retrieve_refund(
         cls,
         fee: str,
         id: str,
-        **params: Unpack["ApplicationFee.RetrieveRefundParams"]
+        **params: Unpack["ApplicationFee.RetrieveRefundParams"],
     ) -> "ApplicationFeeRefund":
         """
         By default, you can see the 10 most recent refunds stored directly on the application fee object, but you can also retrieve details about a specific refund stored on the application fee.
@@ -504,7 +502,7 @@ async def retrieve_refund_async(
         cls,
         fee: str,
         id: str,
-        **params: Unpack["ApplicationFee.RetrieveRefundParams"]
+        **params: Unpack["ApplicationFee.RetrieveRefundParams"],
     ) -> "ApplicationFeeRefund":
         """
         By default, you can see the 10 most recent refunds stored directly on the application fee object, but you can also retrieve details about a specific refund stored on the application fee.
@@ -525,7 +523,7 @@ def modify_refund(
         cls,
         fee: str,
         id: str,
-        **params: Unpack["ApplicationFee.ModifyRefundParams"]
+        **params: Unpack["ApplicationFee.ModifyRefundParams"],
     ) -> "ApplicationFeeRefund":
         """
         Updates the specified application fee refund by setting the values of the parameters passed. Any parameters not provided will be left unchanged.
@@ -548,7 +546,7 @@ async def modify_refund_async(
         cls,
         fee: str,
         id: str,
-        **params: Unpack["ApplicationFee.ModifyRefundParams"]
+        **params: Unpack["ApplicationFee.ModifyRefundParams"],
     ) -> "ApplicationFeeRefund":
         """
         Updates the specified application fee refund by setting the values of the parameters passed. Any parameters not provided will be left unchanged.