Skip to content

Commit e51f3d9

Browse files
authored
FFM-6410 Evaluate multiple and nested prerequisites correctly (#62)
* FFM-6410 Evaluate multiple and nested prerequisites correctly FFM-6410 Evaluate multiple pre-reqs correctly * FFM-6410 Comment * FFM-6410 Use if instead of elif * FFM-6410 1.1.10 release prep
1 parent 6c80ef7 commit e51f3d9

File tree

4 files changed

+6
-5
lines changed

4 files changed

+6
-5
lines changed

featureflags/__init__.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,4 +2,4 @@
22

33
__author__ = """Enver Bisevac"""
44
__email__ = "[email protected]"
5-
__version__ = '1.1.10'
5+
__version__ = '1.1.11'

featureflags/evaluations/evaluator.py

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -295,8 +295,9 @@ def _check_prerequisite(self, parent: FeatureConfig,
295295
if isinstance(variation, Unset) or variation.identifier \
296296
not in pqs.variations:
297297
return False
298-
else:
299-
return self._check_prerequisite(config, target)
298+
# Check for any nested prerequisites
299+
if not self._check_prerequisite(config, target):
300+
return False
300301
return True
301302

302303
def evaluate(self, identifier: str, target: Target) -> Variation:

setup.cfg

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
[bumpversion]
2-
current_version = 1.1.10
2+
current_version = 1.1.11
33
commit = True
44
tag = True
55

setup.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -56,6 +56,6 @@
5656
test_suite="tests",
5757
tests_require=test_requirements,
5858
url="https://github.com/harness/ff-python-server-sdk",
59-
version='1.1.10',
59+
version='1.1.11',
6060
zip_safe=False,
6161
)

0 commit comments

Comments
 (0)