Skip to content

Merge from parent repo#1

Open
pbennett wants to merge 8 commits intopbennett:masterfrom
tmdvs:main
Open

Merge from parent repo#1
pbennett wants to merge 8 commits intopbennett:masterfrom
tmdvs:main

Conversation

@pbennett
Copy link
Copy Markdown
Owner

No description provided.

CMseventeen and others added 8 commits October 19, 2024 17:15
* Update emojidata
* Setup `go.mod`
* Move from `io/ioutil` to `os`
* Remove garbage emoji data entry
* Update build github action
Previously any runes that were less than 4 bytes in length would be
ignored. However, keycap emojis (1️⃣ 2️⃣ 3️⃣ … etc) are prefixed with 2
bytes, eg `31-FE0F-20E3`.

As a fix, I've tweaked the search to skip any runes less than 2 bytes
instead of 4
The [zero-width connector](https://emojipedia.org/zero-width-joiner#technical) is a
special character used to combine multiple emoji into a single emoji
character on screen. For example 👩‍👩‍👧‍👧

This patches the code to skip specific instances of the zero-width
connector to avoid parsing parts of the emoji as unicode
characters/breaking into multiple emoji.

This is based on
#12 (comment)
Introducing some issue templates to the project.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants