Skip to content

Revert local holdout feature gate logic#637

Open
muzahidul-opti wants to merge 1 commit into
masterfrom
muzahidul-opti/enable-local-holdout-feature-gate
Open

Revert local holdout feature gate logic#637
muzahidul-opti wants to merge 1 commit into
masterfrom
muzahidul-opti/enable-local-holdout-feature-gate

Conversation

@muzahidul-opti

@muzahidul-opti muzahidul-opti commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Reverted PR [FSSDK-12394] chore: local holdout logic behind FeatureGates flag #631 (except CLAUDE.md documentation changes)
  • Removed FeatureGates.localHoldouts flag from Constants.swift
  • Unwrapped local holdout evaluation in DefaultDecisionService.swift so it runs unconditionally
  • Removed BaseHoldoutTests.swift and FeatureGateTests_LocalHoldouts.swift
  • Reverted holdout test classes back to inheriting from XCTestCase
  • Removed corresponding project.pbxproj entries for deleted test files

Local holdout evaluation no longer needs to be gated behind a feature flag — it should run by default.

Test plan

  • Existing holdout tests continue to validate local holdout behavior
  • No feature gate flag to toggle; holdout logic is always active

Issues

  • FSSDK-12394

🤖 Generated with Claude Code

Remove the FeatureGates.localHoldouts flag and unwrap local holdout
evaluation so it runs unconditionally, matching the pre-gate behavior.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@muzahidul-opti muzahidul-opti force-pushed the muzahidul-opti/enable-local-holdout-feature-gate branch from 78e291b to c2ca71c Compare June 11, 2026 12:25
@muzahidul-opti muzahidul-opti changed the title Enable local holdout feature gate Revert local holdout feature gate logic Jun 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant