TODOS

[chris-ramon]

TODOs

• {executor/go}: Fix the RootQuery name to match the one from graphql-js. #55
• {validator}: Improves validator.Validate to have support for ignoring order comparison. #53
• {introspection}: Improvements for introspection pkg. #49
• {Introspection}: Implement pkg. #47
• {introspection}: Create scaffolding package of Introspection. #45
• {graphql-js-introspection/index}: Replace index.ts code from TypeScript to JavaScript. #43
• {bin}: Add ./bin/* scripts from compatibility-base. #41
• Adds parsing the specification doc https://github.com/graphql/graphql-spec/releases/tag/October2021 to complement the graphql-js types.
  • Also as a temporal solution let's replace the default header from https://github.com/graphql/graphql-spec/releases/tag/October2021 to point the graphql-js repository.
• Replace top path from graphql-go/... to github.com/graphql-go/....
• Upgrade graphql-go/graphql PR: {types,graphql_test}: Adds Result.ErrorsJoined implementation. graphql#717, so the following code is replaced starting in line: https://github.com/graphql-go/compatibility-standard-definitions/blob/main/executor/go.go#L67-L81.
• Replace types/* interface{} use with another strategy such as generics or later initialization.
• Replace types/* self invoking fields with a working strategy.
• Add bubbletea/bubbletea code comments.
• Update bubbletea/bubbletea dependency, Eg. cmd.New in main.go file.
• Add cmd/cli code comments.
• Add types/* code comments.
• Enable golangci-lint comments.
• Add Unit tests.
• Add Test coverage badges.
• .golangci-lint disable the rule SA9004 because go assumes the type when listing a variables inside a code block using const:

  staticcheck:
    SA9004: false