Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Apply in-code documentation to generated GraphQL #519

Conversation

JairusSW
Copy link
Member

If functions and types are decorated with doc comments (ie. JSDoc for AssemblyScript, GoDoc for Go), then we should be able to have the SDK extract the comments during the build, sanitize them to a common format, and include them in the metadata.

These can then be used during GraphQL schema generation to document the generated API, which should be visible from tools like GraphiQL and Postman.

@JairusSW JairusSW changed the title Use transform to retrieve JSDoc Apply documentation declared in code to generated GraphQL schema Oct 28, 2024
@JairusSW JairusSW changed the title Apply documentation declared in code to generated GraphQL schema Apply in-code documentation to the generated GraphQL Oct 28, 2024
@JairusSW JairusSW changed the title Apply in-code documentation to the generated GraphQL Apply in-code documentation to generated GraphQL Oct 28, 2024
@JairusSW JairusSW marked this pull request as ready for review November 11, 2024 22:58
@JairusSW JairusSW requested a review from a team as a code owner November 11, 2024 22:59
@JairusSW JairusSW enabled auto-merge (squash) November 14, 2024 19:34
@mattjohnsonpint mattjohnsonpint changed the title Apply in-code documentation to generated GraphQL feat: Apply in-code documentation to generated GraphQL Nov 23, 2024
@JairusSW JairusSW merged commit 53aa5a8 into main Nov 23, 2024
86 checks passed
@JairusSW JairusSW deleted the jairus/hyp-2411-extract-doc-comments-from-functions-and-use-in-graphql branch November 23, 2024 19:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants