Skip to content
This repository was archived by the owner on Sep 16, 2022. It is now read-only.

Conversation

@nshahan
Copy link
Contributor

@nshahan nshahan commented Mar 10, 2017

Fixes #296

@nshahan nshahan requested a review from kevmoo March 10, 2017 01:26
@kevmoo
Copy link
Member

kevmoo commented Mar 10, 2017

@nshahan this is enough to fix the issue? Sure?

I'll start the internal merge...

@kevmoo kevmoo self-assigned this Mar 10, 2017
Copy link
Member

@kevmoo kevmoo left a comment

Choose a reason for hiding this comment

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

internal merge is rockin

ferhatb pushed a commit that referenced this pull request Mar 10, 2017
Author: Nicholas Shahan <nshahan@google.com>

Closes #297
Closes #296

-------------
Created by MOE: https://github.com/google/moe
MOE_MIGRATED_REVID=149730411
@ferhatb ferhatb closed this in 23813d6 Mar 10, 2017
@kevmoo kevmoo deleted the fix_common branch March 10, 2017 20:31
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants