Skip to content

Convert non-string type to string before calling tracer.recordBinary#36

Open
neo-wanderer wants to merge 18 commits intoDanielMSchmidt:masterfrom
neo-wanderer:master
Open

Convert non-string type to string before calling tracer.recordBinary#36
neo-wanderer wants to merge 18 commits intoDanielMSchmidt:masterfrom
neo-wanderer:master

Conversation

@neo-wanderer
Copy link
Copy Markdown

No description provided.

@coveralls
Copy link
Copy Markdown

coveralls commented Mar 29, 2018

Coverage Status

Coverage remained the same at 100.0% when pulling ebfdb1b on vigneshnrfs:master into ee2b64e on DanielMSchmidt:master.

@DanielMSchmidt
Copy link
Copy Markdown
Owner

Hey @vigneshnrfs Thank you for your help! CI seems to be red, partially on the integration tests, I will try to fix this soon (#37). If it is ok for you I would like to wait until I can do this, if not I will try to verify it and merge it

@neo-wanderer
Copy link
Copy Markdown
Author

Hi @DanielMSchmidt Let's fix #37 and then do this merge. This can wait.

@neo-wanderer
Copy link
Copy Markdown
Author

This PR also addresses #5

@DanielMSchmidt
Copy link
Copy Markdown
Owner

Could you rebase this branch? I got at least one of the errors in CI fixed, the other one should also be good now

@neo-wanderer
Copy link
Copy Markdown
Author

@DanielMSchmidt It's done. I see just one more failing.

@DanielMSchmidt
Copy link
Copy Markdown
Owner

table flip

I will give it a second try, I think it is green on master because there is no change to push for the website

@neo-wanderer
Copy link
Copy Markdown
Author

@DanielMSchmidt Can we fix the travis-ci issue and merge this?

@neo-wanderer neo-wanderer reopened this Aug 1, 2018
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.

3 participants