You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Right now, if you add a dependency on the emoji-platform-data package, you also get transitive dependencies on emojipedia -> graphql-request -> quite a few GraphQL-related packages. Which is not great. Users who just want the static emoji exports probably don't want all the dependencies for the emojipedia package's Node.js API.
I think it'd be good to rethink how these two packages are organized. Once I finish setting up https://github.com/JoshuaKGoldberg/emoji-blast as a monorepo, I'd like to tackle re-organizing this one. I think there should be a split between code that generates emoji data vs. packages that export the generated data.
Maybe the split could be something like...
@emoji-platform-data/* (e.g. @emoji-platform/data/emojipedia): just the data exports
emoji-platform-data: exports all generation APIs
...I'll need to think more on this.
Additional Info
💖
The text was updated successfully, but these errors were encountered:
Bug Report Checklist
main
branch of the repository.Overview
Right now, if you add a dependency on the
emoji-platform-data
package, you also get transitive dependencies onemojipedia
->graphql-request
-> quite a few GraphQL-related packages. Which is not great. Users who just want the static emoji exports probably don't want all the dependencies for theemojipedia
package's Node.js API.I think it'd be good to rethink how these two packages are organized. Once I finish setting up https://github.com/JoshuaKGoldberg/emoji-blast as a monorepo, I'd like to tackle re-organizing this one. I think there should be a split between code that generates emoji data vs. packages that export the generated data.
Maybe the split could be something like...
@emoji-platform-data/*
(e.g.@emoji-platform/data/emojipedia
): just the data exportsemoji-platform-data
: exports all generation APIs...I'll need to think more on this.
Additional Info
💖
The text was updated successfully, but these errors were encountered: