feat: update emojilib to v3 #132
Conversation
|
|
||
| return results | ||
| }), | ||
| ] |
There was a problem hiding this comment.
I added a few explicit handlers (e.g. :mexico: is no longer an alias of :flag_mexico:). Other changes are causing more test failures. I don't know what to do here - do we maintain explicit backwards compatibility? File issues on emojilib? Accept breaking changes? 😬
cc @omnidan, I don't know what to do 😄
There was a problem hiding this comment.
Talked on Discord: I'm going to add compatibility helpers for the old aliases like this one. That way this isn't a breaking change for users in the sense of dropping emoji name support. Edit: there are a lot of changes 😞 so a compatibility layer would be quite large to write. Never mind on that.
I'll also look into whether we should file any issues on emojilib. muan/emojilib#178 indicated intentionally removing some aliases.
There was a problem hiding this comment.
(I'm waiting on #112 / muan/emojilib#194)
There was a problem hiding this comment.
WHoooo yeah how is the status...
- emojilib v3 had a bunch of changes to its keywords - to the point where node-emoji can't directly transition.
- I pinged the maintainer (muan) and was added as a collaborator there a few weeks ago.
- What I want to do next is make a comparison of keywords: what exists in unicode, common operating systems, common messaging apps, GitHub, etc. (beyond what muan actually asked for)
- I haven't found a good tracking table yet that has all that info
- I sent a request to get access the invite-only Emojipedia API a day or two ago but haven't heard back yet
So there is progress being made, just... slow progress.
There was a problem hiding this comment.
Hello beautiful humans, I love what you did with this lib and thank you for providing this library for developer like me to build on top of it, is there a way we can get this fix merged? I desprately need some emojis which are missing from your lib. Maybe we can add a .extend(":emoji-code","✅") function, just in case some emojis are missing, developer can update it and make it searchable across project?
There was a problem hiding this comment.
Maybe we can add a .extend(":emoji-code","✅") function
Oh! I like that idea standalone - could you file a new issue for that please? Let's track it separately so we can maybe get it in much sooner!
There was a problem hiding this comment.
Filed #144 :)
JoshuaKGoldberg
added
the
status: blocked
Bumps emojilib from the older v2 version we're on to its latest v3.
Note that this contains quite a few changes in how emojis are listed. See https://github.com/omnidan/node-emoji/pull/132/files#r1197993734
Fixes #129. Fixes #131.