-
-
Notifications
You must be signed in to change notification settings - Fork 860
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
Added support for Unicode 12.0 and 13.0 emoji #2143
Added support for Unicode 12.0 and 13.0 emoji #2143
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.
Commits you add to your branch will be added automatically to this PR; no need to close and re-open!
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.
Looks good! I'll run the builds and merge when they're done!
Thank you so much. Sorry, this took a lot more effort than it should have :( |
Sorry to hear it wasn't as simple as expected. We'd have hoped that having the Emoji spreadsheet would make it easier as everything could be copy-pasted between Emojipedia to the spreadsheet and from there to our code. Ideas on how to improve the process? Congratulations on accomplishing it though! If you want, email us at [email protected] and we can send you some merch. |
No, the difficulties were fully on me, I just mostly wasn't careful enough with each of the commits to check other consequences. |
They are now included in the spreadsheet as well as in
emoji.py
. Closes #2081.