Add ruff lint rule - PIE

Author: haydenbaker

packages/smithy-core/src/smithy_core/interceptors.py

@@ -148,7 +148,6 @@ def read_before_execution(
         the `response`. If multiple `read_before_execution` methods throw exceptions,
         the latest will be used and earlier ones will be logged and dropped.
         """
-        pass
 
     def modify_before_serialization(
         self, context: InterceptorContext[Request, None, None, None]
@@ -193,7 +192,6 @@ def read_before_serialization(
         If exceptions are thrown by this hook, execution will jump to
         `modify_before_completion` with the thrown exception as the `response`.
         """
-        pass
 
     def read_after_serialization(
         self, context: InterceptorContext[Request, None, TransportRequest, None]
@@ -214,7 +212,6 @@ def read_after_serialization(
         If exceptions are thrown by this hook, execution will jump to
         `modify_before_completion` with the thrown exception as the `response`.
         """
-        pass
 
     def modify_before_retry_loop(
         self, context: InterceptorContext[Request, None, TransportRequest, None]
@@ -259,7 +256,6 @@ def read_before_attempt(
         exception as the `response` If multiple `read_before_attempt` methods throw
         exceptions, the latest will be used and earlier ones will be logged and dropped.
         """
-        pass
 
     def modify_before_signing(
         self, context: InterceptorContext[Request, None, TransportRequest, None]
@@ -309,7 +305,6 @@ def read_before_signing(
         If exceptions are thrown by this hook, execution will jump to
         `modify_before_attempt_completion` with the thrown exception as the `response`.
         """
-        pass
 
     def read_after_signing(
         self, context: InterceptorContext[Request, None, TransportRequest, None]
@@ -333,7 +328,6 @@ def read_after_signing(
         If exceptions are thrown by this hook, execution will jump to
         `modify_before_attempt_completion` with the thrown exception as the `response`.
         """
-        pass
 
     def modify_before_transmit(
         self, context: InterceptorContext[Request, None, TransportRequest, None]
@@ -384,7 +378,6 @@ def read_before_transmit(
         If exceptions are thrown by this hook, execution will jump to
         `modify_before_attempt_completion` with the thrown exception as the `response`.
         """
-        pass
 
     def read_after_transmit(
         self,
@@ -411,7 +404,6 @@ def read_after_transmit(
         If exceptions are thrown by this hook, execution will jump to
         `modify_before_attempt_completion` with the thrown exception as the `response`.
         """
-        pass
 
     def modify_before_deserialization(
         self,
@@ -467,7 +459,6 @@ def read_before_deserialization(
         If exceptions are thrown by this hook, execution will jump to
         `modify_before_attempt_completion` with the thrown exception as the `response`.
         """
-        pass
 
     def read_after_deserialization(
         self,
@@ -495,7 +486,6 @@ def read_after_deserialization(
         If exceptions are thrown by this hook, execution will jump to
         `modify_before_attempt_completion` with the thrown exception as the `response`.
         """
-        pass
 
     def modify_before_attempt_completion(
         self,
@@ -554,7 +544,6 @@ def read_after_attempt(
         execution will then jump to `read_before_attempt`. Otherwise, execution will
         jump to `modify_before_completion` with the thrown exception as the `response`.
         """
-        pass
 
     def modify_before_completion(
         self,
@@ -605,4 +594,3 @@ def read_after_execution(
         final response. If multiple `read_after_execution` methods throw exceptions,
         the latest will be used and earlier ones will be logged and dropped.
         """
-        pass

packages/smithy-core/src/smithy_core/interfaces/identity.py

@@ -25,7 +25,6 @@ def is_expired(self) -> bool:
 class IdentityProperties(TypedDict):
     """Properties used to help determine the identity to return."""
 
-    ...
 
 
 IdentityPropertiesType = TypeVar("IdentityPropertiesType", bound=IdentityProperties)

packages/smithy-core/src/smithy_core/retries.py

@@ -242,4 +242,3 @@ def refresh_retry_token_for_retry(
 
     def record_success(self, *, token: retries_interface.RetryToken) -> None:
         """Not used by this retry strategy."""
-        pass

packages/smithy-http/src/smithy_http/aio/interfaces/auth.py

@@ -23,7 +23,6 @@
 class SigningProperties(TypedDict):
     """Additional properties loaded to modify the signing process."""
 
-    ...
 
 
 SigningPropertiesType = TypeVar("SigningPropertiesType", bound=SigningProperties)

pyproject.toml

@@ -46,11 +46,11 @@ black = true
 target-version = "py312"
 
 [tool.ruff.lint]
-# candidates: DTZ, EM, INP, ISC, PERF, PIE, RUF, SIM118, SIM401, SLOT
+# candidates: DTZ, EM, INP, ISC, PERF, RUF, SIM118, SIM401, SLOT
 # perhaps in the future: N, PYI, TC, TID
 # probably not, a lot of work: DOC, D, PL, TRY
 
-select = [ "ASYNC", "C4", "E1", "E4", "E7", "E9", "F", "FURB", "G", "I", "LOG", "S", "T", "UP" ]
+select = [ "ASYNC", "C4", "E1", "E4", "E7", "E9", "F", "FURB", "G", "I", "LOG", "PIE", "S", "T", "UP" ]
 exclude = [ "packages/smithy-core/src/smithy_core/rfc3986.py" ]
 
 [tool.ruff.lint.per-file-ignores]