Skip to content

Conversation

@dimaMachina
Copy link
Collaborator

@dimaMachina dimaMachina commented Mar 22, 2023

added changeset explicitly

fixes #2897

@changeset-bot
Copy link

changeset-bot bot commented Mar 22, 2023

🦋 Changeset detected

Latest commit: 2bc2418

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 9 packages
Name Type
graphql-language-service-server Patch
graphql-language-service-cli Patch
codemirror-graphql Patch
@graphiql/toolkit Patch
@graphiql/react Patch
monaco-graphql Patch
vscode-graphql Patch
graphiql Patch
@graphiql/plugin-explorer Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@dimaMachina dimaMachina changed the title [ESLint] enable @typescript-eslint/no-floating-promises [ESLint] enable @typescript-eslint/no-floating-promises, update typescript-eslint to v6 Mar 22, 2023
@dimaMachina dimaMachina force-pushed the no-floating-promises branch from 48cf1a2 to 2bc2418 Compare March 27, 2023 13:22
@dimaMachina dimaMachina marked this pull request as ready for review March 27, 2023 13:32
Copy link
Collaborator

@thomasheyenbrock thomasheyenbrock left a comment

Choose a reason for hiding this comment

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

LGTM, there are a couple of places where we're awaiting now where we did not before, but given the limited context that I have this sounds reasonable 👍

@dimaMachina dimaMachina merged commit 5100700 into graphql:main Apr 14, 2023
@dimaMachina dimaMachina deleted the no-floating-promises branch April 14, 2023 18:21
@acao acao mentioned this pull request Apr 14, 2023
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.

Promise in try/catch

3 participants