Shane Krueger

No, it should be a goal for 4.x, but doesn't need to be any more complete for 4.0.

@sungam3r We only have 160 missing xml comments left. I have updated the list at the top of this issue.

@sungam3r Only classes that relate to GraphQL federation are left @joemcbride If you want to contribute, perhaps you would be interested in adding XML comments to the classes and methods...

@nur858 We are close to releasing v4, which is being developed in the `develop` branch. Could you change your PR base to target `develop` and merge the `develop` branch in...

I do see that `IParseResult` will allow for multiple errors to be returned rather than just a single error. Is that needed by the federated tracing code?

> * Added `src/GraphQL.Federation/readme.md` for installation and configuration instructions. I suggest the documentation be merged in with the main site documentation as follows: - Move & rename the file to...

Might want to add api approvals to the new package. Just add a reference to the new project from GraphQL.ApiTests and then add a reference as an attribute to any...

FYI @nur858 the apiapproval will fail if there's a trailing newline at the end of the apiapproval document. It's a real headache, but that's how it is. Typically I simply...

Unfortunately GitHub requires authentication to access the packages of an open-source repository. Right now it uses the `secrets.GITHUB_TOKEN` token, which is fine when run from the context of `github-dotnet`, but...

@sungam3r I suggest we release the parser now. We have not made any changes to it for quite some time, and I do not foresee making any other changes to...