Skip to content

Commit 8c6e39d

Browse files
committed
Fix all lint issues
1 parent c22bfd9 commit 8c6e39d

File tree

4 files changed

+193
-189
lines changed

4 files changed

+193
-189
lines changed

optimizely/bucketer.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -124,7 +124,7 @@ def bucket(
124124
message = 'Invalid entity key provided for bucketing. Returning nil.'
125125
project_config.logger.debug(message)
126126
return None, []
127-
127+
128128
if isinstance(experiment, dict):
129129
# This is a holdout dictionary
130130
experiment_key = experiment.get('key', '')
@@ -149,7 +149,7 @@ def bucket(
149149
# For experiments, use the existing method
150150
variation = project_config.get_variation_from_id_by_experiment_id(experiment_id, variation_id)
151151
return variation, decide_reasons
152-
152+
153153
# No variation found - log message for empty traffic range
154154
message = 'Bucketed into an empty traffic range. Returning nil.'
155155
project_config.logger.info(message)

optimizely/decision_service.py

Lines changed: 12 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -714,8 +714,10 @@ def get_decision_for_flag(
714714
if not holdout_decision['decision']:
715715
continue
716716

717-
message = f"The user '{user_id}' is bucketed into holdout '{holdout['key']}' " \
718-
f"for feature flag '{feature_flag.key}'."
717+
message = (
718+
f"The user '{user_id}' is bucketed into holdout '{holdout['key']}' "
719+
f"for feature flag '{feature_flag.key}'."
720+
)
719721
self.logger.info(message)
720722
reasons.append(message)
721723
return {
@@ -729,11 +731,11 @@ def get_decision_for_flag(
729731
fallback_result = self.get_variations_for_feature_list(
730732
project_config, [feature_flag], user_context, decide_options
731733
)[0]
732-
734+
733735
# Merge reasons
734736
if fallback_result.get('reasons'):
735737
reasons.extend(fallback_result['reasons'])
736-
738+
737739
return {
738740
'decision': fallback_result.get('decision'),
739741
'error': fallback_result.get('error', False),
@@ -758,7 +760,7 @@ def get_variation_for_holdout(
758760
A DecisionResult for the holdout.
759761
"""
760762
from optimizely.helpers.enums import ExperimentAudienceEvaluationLogs
761-
763+
762764
decide_reasons: list[str] = []
763765
user_id = user_context.user_id
764766
attributes = user_context.get_user_attributes()
@@ -780,7 +782,7 @@ def get_variation_for_holdout(
780782
# Check audience conditions
781783
audience_conditions = holdout.get('audienceIds')
782784
user_meets_audience_conditions, reasons_received = audience_helper.does_user_meet_audience_conditions(
783-
project_config,
785+
project_config,
784786
audience_conditions,
785787
ExperimentAudienceEvaluationLogs,
786788
holdout.get('key', 'unknown'),
@@ -804,8 +806,10 @@ def get_variation_for_holdout(
804806
decide_reasons.extend(bucket_reasons)
805807

806808
if variation:
807-
message = f"The user '{user_id}' is bucketed into variation '{variation['key']}' " \
808-
f"of holdout '{holdout['key']}'."
809+
message = (
810+
f"The user '{user_id}' is bucketed into variation '{variation['key']}' "
811+
f"of holdout '{holdout['key']}'."
812+
)
809813
self.logger.info(message)
810814
decide_reasons.append(message)
811815

0 commit comments

Comments
 (0)