-
Notifications
You must be signed in to change notification settings - Fork 37
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix htmlbody encoding for hebrew text #118
base: master
Are you sure you want to change the base?
Fix htmlbody encoding for hebrew text #118
Conversation
Thanks for your contribution. Could you please add a test case, which would fail without your fix? |
@jugmac00 sure. Working on it. Thanks |
@petri @santoshghimire is a first time contributor, and thus you need to enable the PR to run. GitHub has introduced this restriction to counter bit coin miners who abused CI. I do not have sufficient rights to do this - though I could merge, which is kinda odd. |
I noticed our CI & CodeQL actions had stopped running due to "inactivity". I enabled them. Not sure if that's what you refer to? If not, where do I enable this PR to run then? I looked around but could not see anything. |
The "inactivity" stop, which I think is a very bad term for a stable library, is something unrelated to the crypto restriction. @petri, have a look at the screenshots at https://github.blog/2021-04-22-github-actions-update-helping-maintainers-combat-bad-actors/#new-features-to-help-protect-maintainers I hope this helps. Maybe we encounter some other glitch. In this case it could help when @santoshghimire creates an empty commit and pushes it to trigger CI again:
|
Ok the empty commit did the trick. Weird the approval prompt needed that... |
@santoshghimire As my local test suite on current master works, your changes seem to have introduced a couple of test failures. Could you have a look at the problems? |
@jugmac00 sure. I am on it. |
@jugmac00 I see |
We have dropped Python2 support as you can see from the package description. |
Additionally, we should add a see #119 |
@santoshghimire do you need Python2 support for some reason? |
@petri yes. I have been using 1.3.1 version for python 2.7 |
Fix decoding for binary type
Fixes #117