Skip to content

Commit

Permalink
Merge pull request #14 from mschoenlaub/spec-0.4.0
Browse files Browse the repository at this point in the history
Remove EvaluationOptions from  method signatures
  • Loading branch information
beeme1mr authored Oct 7, 2022
2 parents 0f004af + 059ef20 commit 1727554
Show file tree
Hide file tree
Showing 3 changed files with 0 additions and 14 deletions.
4 changes: 0 additions & 4 deletions open_feature/open_feature_client.py
Original file line number Diff line number Diff line change
Expand Up @@ -210,7 +210,6 @@ def evaluate_flag_details(
key,
default_value,
merged_context,
flag_evaluation_options,
)

after_hooks(type, hook_context, flag_evaluation, merged_hooks, None)
Expand All @@ -237,7 +236,6 @@ def create_provider_evaluation(
key: str,
default_value: typing.Any,
evaluation_context: EvaluationContext = None,
flag_evaluation_options: typing.Any = None,
) -> FlagEvaluationDetails:
"""
Encapsulated method to create a FlagEvaluationDetail from a specific provider.
Expand All @@ -246,15 +244,13 @@ def create_provider_evaluation(
:param key: the string key of the selected flag
:param default_value: backup value returned if no result found by the provider
:param evaluation_context: Information for the purposes of flag evaluation
:param flag_evaluation_options: Additional flag evaluation information
:return: a FlagEvaluationDetails object with the fully evaluated flag from a
provider
"""
args = (
key,
default_value,
evaluation_context,
flag_evaluation_options,
)

if not self.provider:
Expand Down
5 changes: 0 additions & 5 deletions open_feature/provider/no_op_provider.py
Original file line number Diff line number Diff line change
@@ -1,4 +1,3 @@
import typing
from numbers import Number

from open_feature.evaluation_context.evaluation_context import EvaluationContext
Expand All @@ -18,7 +17,6 @@ def get_boolean_details(
key: str,
default_value: bool,
evaluation_context: EvaluationContext = None,
flag_evaluation_options: typing.Any = None,
):
return FlagEvaluationDetails(
key=key,
Expand All @@ -32,7 +30,6 @@ def get_string_details(
key: str,
default_value: str,
evaluation_context: EvaluationContext = None,
flag_evaluation_options: typing.Any = None,
):
return FlagEvaluationDetails(
key=key,
Expand All @@ -46,7 +43,6 @@ def get_number_details(
key: str,
default_value: Number,
evaluation_context: EvaluationContext = None,
flag_evaluation_options: typing.Any = None,
):
return FlagEvaluationDetails(
key=key,
Expand All @@ -60,7 +56,6 @@ def get_object_details(
key: str,
default_value: dict,
evaluation_context: EvaluationContext = None,
flag_evaluation_options: typing.Any = None,
):
return FlagEvaluationDetails(
key=key,
Expand Down
5 changes: 0 additions & 5 deletions open_feature/provider/provider.py
Original file line number Diff line number Diff line change
@@ -1,4 +1,3 @@
import typing
from abc import abstractmethod
from numbers import Number

Expand All @@ -16,7 +15,6 @@ def get_boolean_details(
key: str,
default_value: bool,
evaluation_context: EvaluationContext = EvaluationContext(),
flag_evaluation_options: typing.Any = None,
):
pass

Expand All @@ -26,7 +24,6 @@ def get_string_details(
key: str,
default_value: str,
evaluation_context: EvaluationContext = EvaluationContext(),
flag_evaluation_options: typing.Any = None,
):
pass

Expand All @@ -36,7 +33,6 @@ def get_number_details(
key: str,
default_value: Number,
evaluation_context: EvaluationContext = EvaluationContext(),
flag_evaluation_options: typing.Any = None,
):
pass

Expand All @@ -46,6 +42,5 @@ def get_object_details(
key: str,
default_value: dict,
evaluation_context: EvaluationContext = EvaluationContext(),
flag_evaluation_options: typing.Any = None,
):
pass

0 comments on commit 1727554

Please sign in to comment.