Skip to content

Conversation

aliabbasrizvi
Copy link
Contributor

Summary

Thanks to @emielrv for reporting and figuring out the issue

Issues

Fixes #220

@coveralls
Copy link

Coverage Status

Coverage remained the same at 97.646% when pulling 31b9fd4 on ali/fix_encoding into d3ffe33 on master.

4 similar comments
@coveralls
Copy link

Coverage Status

Coverage remained the same at 97.646% when pulling 31b9fd4 on ali/fix_encoding into d3ffe33 on master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 97.646% when pulling 31b9fd4 on ali/fix_encoding into d3ffe33 on master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 97.646% when pulling 31b9fd4 on ali/fix_encoding into d3ffe33 on master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 97.646% when pulling 31b9fd4 on ali/fix_encoding into d3ffe33 on master.

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

@aliabbasrizvi aliabbasrizvi merged commit 9fee8ff into master Dec 5, 2019
@aliabbasrizvi aliabbasrizvi deleted the ali/fix_encoding branch December 5, 2019 16:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
4 participants