Merge pull request #780 from opsmill/pog-PLW1641

Add __hash__ method to object that already has __eq__

Author: ogenstad

infrahub_sdk/recorder.py

@@ -34,6 +34,9 @@ def default(cls) -> NoRecorder:
     def __eq__(self, other: object) -> bool:
         return isinstance(other, NoRecorder)
 
+    def __hash__(self) -> int:
+        return hash(self.__class__)
+
 
 class JSONRecorder(BaseSettings):
     model_config = SettingsConfigDict(env_prefix="INFRAHUB_JSON_RECORDER_")

pyproject.toml

@@ -247,7 +247,6 @@ ignore = [
     "PLR2004", # Magic value used in comparison
     "PLR6301", # Method could be a function, class method, or static method
     "PLW0603", # Using the global statement to update `SETTINGS` is discouraged
-    "PLW1641", # Object does not implement `__hash__` method
     "RUF005",  # Consider `[*path, str(key)]` instead of concatenation
     "RUF029",  # Function is declared `async`, but doesn't `await` or use `async` features.
     "S311",    # Standard pseudo-random generators are not suitable for cryptographic purposes