Drop babel-runtime

[dustinfarris]

Relying on this library can be problematic depending on the client environment. It is definitely problematic for me.

GraphQL's justification does a better job of explaining than I ever could—so I am going to quote their commit message from graphql/graphql-js@188881b

Removes depencency on babel-runtime.

Simplifies babel transform using "loose" options and removing some transforms. This results in a subset of ES2015 environment, but this source has been pretty careful to avoid the more novel features in order to support more environments.

The result for those using graphql via npm is fewer dependencies and less reliance on a modern JS environment, both good things when wanting to use graphql as a depencency in a client-side tool, like Apollo or Relay.

Note that the above change took effect in graphql 0.6.1. (cashay currently pegs graphql 0.6.0)

[dustinfarris]

@mattkrick do you have any thoughts on this?

[mattkrick]

yeah, you're absolutely right! I need to dig into this & check out the easiest way to do that.
I was hoping await/async would have landed sooner in browsers/node so I could just rip out the polyfill, but i suppose this is the better way in case someone is still using IE10.

[dustinfarris]

That's great, I'm glad you're onboard. I wonder if swapping async/await with good 'ol promises would be ok? (albeit much uglier)

As soon as we can drop babel-runtime I have an awesome demo project to put online a la ember-cashay.

[mattkrick]

yeah, that's not a terrible idea, especially since there are only 2 of them. https://github.com/MatAtBread/fast-async is another option

[dustinfarris]

@mattkrick are you working on this? If not I will take a stab at it.

[mattkrick]

Not right now, I got a work sprint that's kicking my butt at the moment. If
you got a PR in mind by all means! Otherwise it may be another week or 2.

On Nov 18, 2016 12:49 PM, "Dustin Farris" notifications@github.com wrote:

@mattkrick https://github.com/mattkrick are you working on this? If not
I will take a stab at it.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#138 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFQjvwVnhSVvWIXmC0tvycLK6yJ_JlFqks5q_g9dgaJpZM4KnxXv
.