Frederick Zhang

Results 190 comments of Frederick Zhang

Logs: ``` {"level":"debug","message":"host.start"} {"level":"info","message":"handleRequest: "} {"level":"debug","message":"request received: "} {"level":"info","message":"handleNotification: "} {"level":"debug","message":"host.handlePlugin: "} {"level":"debug","message":"createPlugin./home/frederick/.vim/plugged/todoist.nvim/rplugin/node/todoist.clearCache: true"} {"level":"debug","message":"getPlugin.alwaysInit"} {"level":"debug","message":"request -> neovim.api.nvim_eval"} {"level":"debug","message":"response -> neovim.api.nvim_eval: [object Object]"} {"level":"debug","message":"request -> neovim.api.nvim_call_function"} {"level":"debug","message":"response -> neovim.api.nvim_call_function: -1"}...

@kobylynskyi Thanks for your quick response! I'm not very familiar this project so please correct me if I'm wrong, but if we are going to track explicitly set fields in...

I tried out the `Optional` solution and it seems to be roughly ok: https://github.com/Frederick888/graphql-java-codegen/commit/50d7ecdc625bbdda9a93f05c494ffc1b6bcc19c4 @kobylynskyi What do you think?

@robbertnoordzij Thanks for the comment, that makes sense. Second try: https://github.com/Frederick888/graphql-java-codegen/commit/c449cb2956a0eb1560f278103090e9634aee8d82 A few questions: 1. Not sure if we should mark both `jackson-databind` and `jackson-datatype-jdk8` as `providedCompile`? I'm not terribly...

By the way I just realised that this seems to be in GraphQL specs: https://github.com/graphql/graphql-spec/pull/83 Related graphql-java PR: https://github.com/graphql-java/graphql-java/pull/156

@robbertnoordzij You were right. The PR closed https://github.com/graphql-java/graphql-java/issues/106 but only solved the issue partially (IIUC it omits null fields which haven't got default values from the final result, but provides...

@robbertnoordzij By the way, what were your thoughts on my second try?

@robbertnoordzij Hmmm... which part are you referring to? In my second try it's not using `Optional` for mandatory fields, unless I overlooked something? For example, in the following generated model,...

Hmm... It's seems that this is causing problems. The latest snapshot doesn't work: > [coc.nvim]: Uncaught exception: Header must provide a Content-Length property. Manually compiled `v0.66.0` and it's ok now....

@dansomething I was using the same snapshot of jdt.ls with OpenJDK 11.0.9 under macOS and I had this issue with two projects. Those two projects have got similar dependencies though...