search

mattkrick / cashay

:moneybag: Relay for the rest of us :moneybag:
MIT License
453 stars 28 forks source link

Drop graphql #148

Closed dustinfarris closed 7 years ago

dustinfarris commented 7 years ago

Leaving this as a placeholder. I plan on attempting this eventually, if no one does it first.

dustinfarris commented 7 years ago

@mattkrick do you have any further thought on this? from a previous note,

Currently, aside from a handful of string constants, cashay just uses parse and print.

The constants are easy enough to copy, but print is more involved, and parse even more so.

mattkrick commented 7 years ago

the naive approach would basically be a cut n paste of both of those functions & deps + string constraints. it'll basically be the entire /language folder of graphql.

the smarter way would be to build cashay via webpack & perform tree shaking there. that way, the cashay bundle would only include the pieces of graphql that it needed. you could output 2 files (commonjs/umd) if needed. http://www.2ality.com/2015/12/webpack-tree-shaking.html

dustinfarris commented 7 years ago

ok, using webpack now (just for cashay/graphql) — thanks for the link