Skip to content
This repository has been archived by the owner on Jan 11, 2024. It is now read-only.

Feature variable APIs #106

Merged
merged 2 commits into from Jan 19, 2018
Merged

Feature variable APIs #106

merged 2 commits into from Jan 19, 2018

Conversation

mjc1283
Copy link
Contributor

@mjc1283 mjc1283 commented Jan 19, 2018

No description provided.

@optibot
Copy link

optibot commented Jan 19, 2018

Can one of the admins verify this patch?

1 similar comment
@optibot
Copy link

optibot commented Jan 19, 2018

Can one of the admins verify this patch?

@coveralls
Copy link

coveralls commented Jan 19, 2018 •
edited

Coverage Status

Coverage increased (+0.1%) to 98.737% when pulling 53136f5 on mcarroll/feature-variable-apis into 8c3ee5b on master.

@mikeproeng37
Copy link
Contributor

build

mikeproeng37
mikeproeng37 approved these changes Jan 19, 2018
View reviewed changes
Copy link
Contributor

@mikeproeng37 mikeproeng37 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm, please merge master in and wait for e2e tests to pass

@coveralls
Copy link

coveralls commented Jan 19, 2018 •
edited

Coverage Status

Coverage increased (+0.1%) to 98.737% when pulling 5027890 on mcarroll/feature-variable-apis into 8c3ee5b on master.

@wangjoshuah
Copy link
Contributor

build

@mjc1283 mjc1283 merged commit c29c01f into master Jan 19, 2018
@mjc1283 mjc1283 deleted the mcarroll/feature-variable-apis branch January 19, 2018 19:51
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@mikeproeng37 mikeproeng37 mikeproeng37 approved these changes

@aliabbasrizvi aliabbasrizvi Awaiting requested review from aliabbasrizvi

@thomaszurkan-optimizely thomaszurkan-optimizely Awaiting requested review from thomaszurkan-optimizely

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@mjc1283 @optibot @coveralls @mikeproeng37 @wangjoshuah