update to graphql-js@0.5

[jonathanglasmeyer]

graphql-js@0.5 has changed the params of the resolve fns -- info is now 4th, context is now 3rd. The resolver will have to be updated in order to be compatible with this breaking change.

[mickhansen]

#205

[mickhansen]

Looks like graphql-relay finally got around to updating to graphql, so i'll finish the PR :)

[mickhansen]

Or no, they haven't released a new version, so still waiting on them.

[jonathanglasmeyer]

! cool, missed that. thanks for the pointer!
btw: we don't use relay and it's a bit weird that is still is a required dependency. But i guess there is no better way to do this, is it?

[mickhansen]

Yeah it's not ideal, but haven't found a way to solve it.
With moving to npm3 in the future we can make both graphql and graphql-relay peer dependencies probably.

[jonathanglasmeyer]

npm would still warn when i don't install graphql-relay though, right? I don't now if there's a way to specify "optional peerDependencies" ?

[mickhansen]

Hmm, good question, i don't know if it warns or errors.

[satya164]

@mickhansen NPM 2 errors, NPM 3 warns.

[mickhansen]

Thanks @satya164, sounds like the way to go moving forward.

[mickhansen]

https://github.com/mickhansen/graphql-sequelize/releases/tag/v1.0.0

[jonathanglasmeyer]

👍 Congrats!