Skip to content

Conversation

miorel
Copy link
Contributor

@miorel miorel commented Aug 23, 2024

This allows deleting several files.

The new approach fails to print non-scalar default values, but other than that it's the same output as the old schema. Will bring this back in another commit.

omFavoriteList(
data: OMFavoriteListInput! = { skip: 0, limit: 20, sort_dsc: true }
): OMFavoriteListNode!
omFavoriteList(data: OMFavoriteListInput!): OMFavoriteListNode!
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The new approach fails to print non-scalar default values, but other than that it's the same output as the old schema. Will bring this back in another PR.

@miorel miorel changed the title use graphql fully LeetCode GraphQL: fully use core graphql for printing the schema Aug 23, 2024
@miorel miorel requested review from simona1 and elimanzo August 23, 2024 14:43
@miorel miorel marked this pull request as ready for review August 23, 2024 14:43
@miorel miorel merged commit d8b4be6 into main Aug 24, 2024
6 checks passed
@miorel miorel deleted the graphql-full branch August 24, 2024 07:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants