graphql@16 dropped long-deprecated support for positional arguments

[nkahnfr]

Hello,

As indicated in the title, graphql v16 introduced a breaking change by removing support for positional args in graphql/execute/subscribe functions (graphql/graphql-js#2904).

Do you plan to update DDP-Apollo to support the latest version of GraphQL.js?

It seems quite a straightforward task so I can create a pull request if it helps.
I identified two files to update so far:

• ddp-apollo/src/createExecutor.js

  Line 24 in 838abfa

  return execute(

• ddp-apollo/src/createGraphQLPublication.js

  Line 44 in 838abfa

  .then((completeContext) => subscribe(

Passing an object with named arguments is supported for a long time so there should be no compatibility issue with previous releases of GraphQL.js (I checked v14.0.0 and it's ok).

Thanks for your help.

[Cliffzz]

Thanks for mentioning this, just released a new version so it should work now.