Fix unicode conversion issue.

cameronmaske / celery-once

Celery Once allows you to prevent multiple execution and queuing of celery tasks.

https://pypi.python.org/pypi/celery_once/

BSD 2-Clause "Simplified" License

661 stars 90 forks source link

Fix unicode conversion issue. #73

Closed asyncmind0 closed 4 years ago

coveralls commented 6 years ago

Pull Request Test Coverage Report for Build 205

1 of 1 (100.0%) changed or added relevant line in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.6%) to 96.104%

Files with Coverage Reduction	New Missed Lines	%
celery_once/helpers.py	1	95.0%
<!--	Total:	1		-->

Totals
Change from base Build 203:	-0.6%
Covered Lines:	148
Relevant Lines:	154

💛 - Coveralls

jheld commented 6 years ago

Is this a legit test failure? I saw the output...looked like maybe it was supposed to skip?

Is it failing on force_text or six.text_type? Because it looks like maybe we could specify the encoding to utf-8 and it would fix it? Or is it a different problem?

cameronmaske commented 5 years ago

@jagguli Sorry for being so silent on this PR, but could you add more context to the issue you are running into, that this solves?

cameronmaske commented 4 years ago

Closing this due to being stale.