FSharp.Data.GraphQL icon indicating copy to clipboard operation
FSharp.Data.GraphQL copied to clipboard

Complete the AST according to the June 2018 GraphQL spec edition

Open haf opened this issue 6 years ago • 5 comments

  • Add some code-docs so that developers using the AST can read about what they are doing
  • Add TypeSystemDefinition and TypeSystemExtension to the Definition DU
  • Move DirectiveLocation from TypeSystem to Ast
  • Update usages

haf avatar Sep 04 '19 17:09 haf

Is this wanted?

haf avatar Sep 26 '19 14:09 haf

Hi @haf

I sent you a message on gitter a few days ago. We're definitely wanting this PR, but since this is a breaking change we're waiting until we complete the other 2.0 changes in. Thanks, John

johnberzy-bazinga avatar Sep 26 '19 17:09 johnberzy-bazinga

@haf is it relevant? If so, could you rebase this and make mergable?

xperiandri avatar May 21 '22 12:05 xperiandri

Relevant: Yes, IMO without seeing what work of mine that authors of this repo duplicated.

Can rebase: no, sorry, not without payment. The code is as-is and I'm fine with it being rejected or merged, whichever suits you best.

haf avatar May 23 '22 07:05 haf

@haf, pinged you in F# Slack

xperiandri avatar Aug 14 '22 09:08 xperiandri