-
Notifications
You must be signed in to change notification settings - Fork 4
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
New release on opam #6
Comments
@zapashcanon @Swrup The fork looks exciting with all the (sub)categories! I see that we all noticed some of the same issues (e.g. diacritics) but fixed them slightly differently, and in some cases I love your solution (e.g., parsing the official data files), and in some cases I like mine better (e.g., using "first" instead of "_1st"). How should I join the party? Should I wait for the PR? |
Hi @favonia, sorry I should have made a PR earlier. The thing is we wanted to only give the gencode and not the generated emoji.ml Also I would like to have all pictographs (maybe in another lib). |
@Swrup What's missing in your fork now? I am interested in helping if it only needs some easy fix. I do think in the long run we should parse the data files instead of scraping the web pages, and uucd is a good library. (Not sure about uucp because emojis are spread across so many blocks.) Anyways, looking forward to your PR. |
@zapashcanon @Swrup Any progress on this? |
Sorry, I don't remember what was left to do. I think @Swrup had something working and it just need to be published on opam. |
Hello, My fork works with the unicode v15 published in September. The code to generate emoji.ml parses files from unicode.org: http://www.unicode.org/emoji/charts/emoji-list.html and https://www.unicode.org/emoji/charts/full-emoji-modifiers.html but it conflict with the lasts commits and also emoji.ml is still not generated at compilation (because It's too slow) and some dislike this |
I noted you switched to Dune, it makes sense to upload a new release to the opam.
The text was updated successfully, but these errors were encountered: