-
-
Notifications
You must be signed in to change notification settings - Fork 32.1k
gh-62824: add alias for iso-8859-8-i and -e to iso_8859_8 #32279
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ran 266 tests in 4.734s
OK (skipped=13)
Looks ok.
@malemburg (as an encodings expert) |
Will this PR be accepted? Why is it still open? |
Yes, I'd accept the PR, but before doing so, the conflicts will have to be resolved and the tests need to finish without errors. |
Closing in favor of #134306. |
Looks like you get quick results your way! |
Add aliases for iso-8859-8-i and iso-8859-8-e, which is the same as iso-8859-8
Co-authored-by: Ben Galin (bensws)
Source for this PR is derived from a patch submitted by bensws:
https://bugs.python.org/file35736/8859-8_aliases_and_test.patch
https://bugs.python.org/issue18624