-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Add withExperiment extension method to Nimbus #16926
Add withExperiment extension method to Nimbus #16926
Conversation
a309c47
to
c2df473
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
* Short-hand for `mozilla.components.service.nimbus.NimbusApi.getExperimentBranch`. | ||
*/ | ||
fun NimbusApi.withExperiment(experimentId: String) = | ||
this.getExperimentBranch(experimentId) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would those extension be something we could ship with the nimbus library?
c2df473
to
b64ec17
Compare
Codecov Report
@@ Coverage Diff @@
## master #16926 +/- ##
============================================
- Coverage 30.89% 30.88% -0.02%
Complexity 1240 1240
============================================
Files 453 454 +1
Lines 18519 18527 +8
Branches 2583 2584 +1
============================================
Hits 5722 5722
- Misses 12338 12346 +8
Partials 459 459
Continue to review full report at Codecov.
|
b64ec17
to
28522d5
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
Fixes #16925.
Adds extension method, then re-implements the existing experiments in the terms of those extension methods.