Skip to content

Conversation

@chamons
Copy link
Contributor

@chamons chamons commented Oct 21, 2016

@chamons
Copy link
Contributor Author

chamons commented Oct 21, 2016

Similar in spirit to #1014 but should fix both XI and XM since logic is shared in this case.

@chamons
Copy link
Contributor Author

chamons commented Oct 21, 2016

@spouliot Also proposed for C9.
@jstedfast Please review

@chamons
Copy link
Contributor Author

chamons commented Oct 21, 2016

This patch assumes PR 1014 lands first, It might not build w\o the other test changes (I just cherry picked for review).

@jstedfast
Copy link
Member

looks good

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.

4 participants