chore: remove excluded ruff rules and fix issues (#32)

remove excluded ruff rules and fix issues

Signed-off-by: gruebel <anton.gruebel@gmail.com>

openfeature/contrib/hooks/otel.py

@@ -1,8 +1,9 @@
 import json
 
+from opentelemetry import trace
+
 from openfeature.flag_evaluation import FlagEvaluationDetails
 from openfeature.hook import Hook, HookContext
-from opentelemetry import trace
 
 OTEL_EVENT_NAME = "feature_flag"
 

openfeature/contrib/providers/flagd/__init__.py

@@ -1,3 +1,3 @@
 from .provider import FlagdProvider
 
-FlagdProvider
+__all__ = ["FlagdProvider"]

openfeature/contrib/providers/flagd/provider.py

@@ -26,6 +26,7 @@
 
 import grpc
 from google.protobuf.struct_pb2 import Struct
+
 from openfeature.evaluation_context import EvaluationContext
 from openfeature.exception import (
     FlagNotFoundError,
@@ -173,12 +174,12 @@ def _resolve(
             message = f"received grpc status code {code}"
 
             if code == grpc.StatusCode.NOT_FOUND:
-                raise FlagNotFoundError(message)
+                raise FlagNotFoundError(message) from e
             elif code == grpc.StatusCode.INVALID_ARGUMENT:
-                raise TypeMismatchError(message)
+                raise TypeMismatchError(message) from e
             elif code == grpc.StatusCode.DATA_LOSS:
-                raise ParseError(message)
-            raise GeneralError(message)
+                raise ParseError(message) from e
+            raise GeneralError(message) from e
 
         # Got a valid flag and valid type. Return it.
         return FlagEvaluationDetails(

pyproject.toml

@@ -64,10 +64,6 @@ select = [
 ]
 ignore = [
     "E501", # the formatter will handle any too long line
-    # all following rules will be removed in the next PR
-    "I001",
-    "B018",
-    "B904",
 ]
 preview = true
 

tests/hooks/test_otel.py

@@ -1,13 +1,13 @@
 from unittest.mock import Mock
 
 import pytest
-from openfeature.evaluation_context import EvaluationContext
-from openfeature.flag_evaluation import FlagEvaluationDetails, FlagType
-from openfeature.hook import HookContext
 from opentelemetry import trace
 from opentelemetry.trace import Span
 
 from openfeature.contrib.hooks.otel import TracingHook
+from openfeature.evaluation_context import EvaluationContext
+from openfeature.flag_evaluation import FlagEvaluationDetails, FlagType
+from openfeature.hook import HookContext
 
 
 @pytest.fixture