Skip to content
This repository has been archived by the owner on Mar 20, 2023. It is now read-only.

Fix RequestInfo typings #646

Merged
merged 1 commit into from
Jun 28, 2020

Conversation

danielrearden
Copy link
Collaborator

There were two other properties that I missed as well 馃槗

@danielrearden danielrearden changed the title Fix RequestInfo TS type Fix RequestInfo typings Jun 27, 2020
@@ -171,7 +171,7 @@ export type RequestInfo = {|
/**
* The result of executing the operation.
*/
result: ExecutionResult | null,
result: ExecutionResult,
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Removed the null here because result could never be null (see here)

Copy link
Member

Choose a reason for hiding this comment

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

Thanks for comment 馃憤
It's very helpful.

@IvanGoncharov IvanGoncharov merged commit 44e8ce6 into graphql:master Jun 28, 2020
@IvanGoncharov IvanGoncharov added the PR: bug fix 馃悶 requires increase of "patch" version number label Jun 28, 2020
@danielrearden danielrearden deleted the fix-request-info branch June 28, 2020 14:23
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
PR: bug fix 馃悶 requires increase of "patch" version number
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants