Benedikt Franke

I like option 2, since we can do that without a breaking change. Can you create a pull request for that? You can add a TODO to simplify and bump...

I have never used PHP on Windows and I have no plans to do it, so supporting it is really low on my list of priorities. If you can provide...

> I think it would also make sense that the relations.column type will default to String if column enumerations are missing, instead of trying to resovle the column enum type...

To clarify: In your example, the field with `@hasMany` is nullable. Are you talking about differences between handling an empty list or `null` there? I think it should most likely...

We generally avoid nullable lists in our GraphQL schema, my company's coding guidelines say this: Fields that return lists SHOULD be non-nullable and contain non-nullable elements. For clients, it is...

I just found that Lighthouse already implements validation caching, see https://github.com/nuwave/lighthouse/pull/2603. In general, the approach to hashing to schema and hashing the query string is the same, but there are...

> Not sure I follow. I don't use Lighthouse, and I'm not familiar with it. What does it have to do with what I'm doing in this PR? Lighthouse is...

I think this issue is basically the same as https://github.com/barryvdh/laravel-ide-helper/issues/1585, and it is a bit more general. The presence of any macros on `Collection` will cause it to be included...