Benedikt Franke

Feel free to try your hands at a PR. Some pointers: We need to allow directives to apply to interface fields as well as object type fields: https://github.com/nuwave/lighthouse/blob/ab80dd8f2b7697df1d85114670c7d23f04945a05/src/Schema/AST/ASTBuilder.php#L285 For backwards...

Looks like this will be possible in an upcoming version of the GraphQL specification - see https://github.com/graphql/graphql-spec/pull/805#issuecomment-1029319320 We have to implement this in the base library first though, see https://github.com/webonyx/graphql-php/issues/110

Unless you have somebody DDOS'ing your server, I would expect invalid queries not to be repeated often. Thus, we could just cache queries that have executed successfuly, and then just...

The fix for this depends on https://github.com/webonyx/graphql-php/pull/1068 which will be available in graphql-php 15. If you are brave, you can try running https://github.com/nuwave/lighthouse/pull/1898 - it will be available in Lighthouse...

Good find! I am fine with requiring certain minor/bugfix versions of dependencies in a minor release of Lighthouse. If you can make the minimal required bumps that would be great.

We can just skip the test for `DateTimeUtc` in Laravel 5.6, we do something similar in a bunch of places.

Thanks @olivernybroe for your work on this. I included parts of this in https://github.com/nuwave/lighthouse/pull/1898, but do not plan to make potentially breaking dependency changes in this version.

To enable this, we would have to pass the resolver arguments - in your case `$root` - to `ArgumentSet::enhanceBuilder()` and all functions below it. Such a change would be a...

If we were to change this, we should also pass along the other resolver arguments, not just `$root`. That way we won't have to extend this even further if other...

I am confused what behaviour you would like out of `@scope`. Something about depending on arguments, but also using it on a query field (as opposed to a mutation field?)?...