Liam Stevenson

Results 11 comments of Liam Stevenson

> Thanks for the PR. Do you think we could add a test to show this use-case? Would https://github.com/sangria-graphql/sangria/blob/main/modules/derivation/src/test/scala/sangria/streaming/StreamSpec.scala be a good place for that? I went ahead and added...

Or an even better solution would probably be creating some sort of `SubscriptionFieldResolver` to complement `FieldResolver`, which could deal with the tagging.

I've been running into the same issue and believe that it's a bug. Did you find a way around this?

@yanns I've looked into the issue more, and I'm convinced it is a bug with sangria. I created a test case for it on my own fork, [located here](https://github.com/liam923/sangria/blob/nested-input-array/modules/core/src/test/scala/sangria/marshalling/FromInputSpec.scala#L640). I...

This is great, it seems to fix the bug I originally reported. Thanks for the quick turnaround. I did find a new issue that it causes, however. Once some wrapping...

If I understand this issue correctly, I think the heuristic from https://github.com/ocaml/merlin/pull/1882 would help here. That heuristic only kicks in when there are multiple files with a matching digest. But...

Internally at Jane Street, we're planning on changing the build rules so that the compiler records the relative path to the source file from the project root. And then Merlin...

> @liam923 I am not sure your proposition would fix the issue here, but I happy to try a POC if you have one :-) Yes I agree, I've realized...

> Also note that the bootstrap is not necessary in this case. Ah thanks, sorry about that. I must've run a stray command.