-
Notifications
You must be signed in to change notification settings - Fork 0
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/use monaco: TypeScript fixups #18
Conversation
export namespace GraphiQL { | ||
export type GetDefaultFieldNamesFn = (type: GraphQLType) => string[]; | ||
export type Maybe<T> = T | null | undefined; | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Whoops, I think I need to put this back?
@@ -5,118 +5,111 @@ | |||
* LICENSE file in the root directory of this source tree. | |||
*/ | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
} | ||
} | ||
return result; | ||
// const { insertions, result } = fillLeafs( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ah yes! i think this is where we get the auto-expanding fields? trying to figure out how to re-enable that in monaco today
@cshaver I can port over a good portion of these changes to the current RFC branch in a PR, there's some very useful improvements here! |
@@ -11,13 +11,13 @@ import type { | |||
Diagnostic, | |||
CompletionItem as GraphQLCompletionItem, | |||
} from 'graphql-language-service-types'; | |||
import type monaco from 'monaco-editor'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it's available globally across the package now, so no need for this! i just need to add it to eslint globals
@cshaver I think I captured most of these improvements, thank you! |
To be merged into #17.
Doing some Sunday afternoon typescript fixes. I was curious about a couple
@ts-ignore
s and got carried away, so I tried to resolve all the ones I could, plus a couple spelling edits and year updates along the way.