Skip to content

Conversation

@karmel
Copy link
Contributor

@karmel karmel commented Mar 20, 2019

The test itself is failing right now as per the comments in #68 , but here are the badge links for the Readme when ready.

@facaiy
Copy link
Member

facaiy commented Mar 20, 2019

Karmel, I cannot reproduce the failure. What image are you using? And could you print the path before opening it?

@facaiy
Copy link
Member

facaiy commented Mar 22, 2019

Gently ping @karmel @av8ramit

@av8ramit
Copy link
Contributor

@facaiy sorry I did not have cycles to look into this this week. Would you like the exact commands being run internally?

@facaiy
Copy link
Member

facaiy commented Mar 23, 2019

Amit, would you mind taking a look #68?

@av8ramit
Copy link
Contributor

@facaiy I fixed it.

@av8ramit
Copy link
Contributor

You can merge this now @karmel the preview looks good.

Copy link
Member

@seanpmorgan seanpmorgan left a comment

Choose a reason for hiding this comment

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

Looks great.. thank you both!

@seanpmorgan seanpmorgan merged commit 3b7293d into master Mar 26, 2019
@seanpmorgan seanpmorgan deleted the sanity-check branch March 26, 2019 19:28
@karmel
Copy link
Contributor Author

karmel commented Mar 26, 2019 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants