Apply B rule

Author: chezou

pyproject.toml

@@ -54,7 +54,7 @@ select = [
   "F",
   "I",
   "UP",
-  # "B",
+  "B",
 ]
 exclude = ["tdclient/test/*"]
 ignore = ["E203", "E501"]

tdclient/api.py

@@ -314,7 +314,9 @@ def post(
                 urllib3.exceptions.PoolError,
             ):
                 if not self._retry_post_requests:
-                    raise APIError("Retrying stopped by retry_post_requests == False")
+                    raise APIError(
+                        "Retrying stopped by retry_post_requests == False"
+                    ) from None
 
             if cumul_retry_delay <= self._max_cumul_retry_delay:
                 log.warning(
@@ -402,7 +404,7 @@ def put(
             urllib3.exceptions.TimeoutError,
             urllib3.exceptions.PoolError,
         ):
-            raise APIError(f"Error: {repr(response)}")
+            raise APIError(f"Error: {repr(response)}") from None
 
         log.debug(
             "REST PUT response:\n  headers: %s\n  status: %d\n  body: <omitted>",
@@ -570,7 +572,7 @@ def checked_json(self, body: bytes, required: list[str]) -> dict[str, Any]:
         try:
             js = json.loads(body.decode("utf-8"))
         except ValueError as error:
-            raise APIError(f"Unexpected API response: {error}: {repr(body)}")
+            raise APIError(f"Unexpected API response: {error}: {repr(body)}") from error
         js = dict(js)
         if 0 < [k in js for k in required].count(False):
             missing = [k for k in required if k not in js]

tdclient/job_model.py

@@ -112,6 +112,7 @@ def update(self) -> None:
     def _update_status(self) -> None:
         warnings.warn(
             "_update_status() will be removed from future release. Please use update() instaed.",
+            stacklevel=2,
             category=DeprecationWarning,
         )
         self.update()

tdclient/util.py

@@ -43,6 +43,7 @@ def validate_record(record: Record) -> bool:
     if not any(k in record for k in ("time", b"time")):
         warnings.warn(
             'records should have "time" column to import records properly.',
+            stacklevel=2,
             category=RuntimeWarning,
         )
     return True
@@ -132,7 +133,7 @@ def merge_dtypes_and_converters(
                     "Unrecognized dtype {!r}, must be one of {}".format(
                         dtype, ", ".join(repr(k) for k in sorted(DTYPE_TO_CALLABLE))
                     )
-                )
+                ) from None
     if converters is not None:
         for column_name, parse_fn in converters.items():
             our_converters[column_name] = parse_fn