-
Notifications
You must be signed in to change notification settings - Fork 160
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
FFs can now be toggled in release builds too #3101
FFs can now be toggled in release builds too #3101
Conversation
📱 Scan the QR code below to install the build (arm64 only) for this PR. |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #3101 +/- ##
===========================================
+ Coverage 75.96% 76.00% +0.03%
===========================================
Files 1636 1635 -1
Lines 38617 38600 -17
Branches 7463 7461 -2
===========================================
+ Hits 29335 29337 +2
+ Misses 5393 5373 -20
- Partials 3889 3890 +1 ☔ View full report in Codecov by Sentry. |
- Removed `StaticFeatureFlagProvider`. - Always provide `PreferencesFeatureFlagProvider`. - For the default values of feature flags, use a lambda with a `BuildMeta` parameter so we can customize the return value based on its data.
9d50729
to
a4872dc
Compare
Quality Gate passedIssues Measures |
Type of change
Content
StaticFeatureFlagProvider
.PreferencesFeatureFlagProvider
.BuildMeta
parameter so we can customize the return value based on its data.Motivation and context
Fixes #3073.
Tests
Tested devices
Checklist