apollographql / apollo-client

:rocket:  A fully-featured, production ready caching GraphQL client for every UI framework and GraphQL server.
https://apollographql.com/client
MIT License
19.34k stars 2.66k forks source link

Type `MutationResult.reset` as an arrow function #12027

Closed JavaScriptBach closed 1 month ago

JavaScriptBach commented 1 month ago

Type Mutation.reset as an arrow function.

This function doesn't depend on this, so it can be safely passed around as value. Typing this as an arrow function makes it clear and plays nicer with https://typescript-eslint.io/rules/unbound-method/

apollo-cla commented 1 month ago

@JavaScriptBach: Thank you for submitting a pull request! Before we can merge it, you'll need to sign the Apollo Contributor License Agreement here: https://contribute.apollographql.com/

netlify[bot] commented 1 month ago

Deploy Preview for apollo-client-docs ready!

Name Link
Latest commit d9ab0d1b4f3c419f63093527eac746fa32795812
Latest deploy log https://app.netlify.com/sites/apollo-client-docs/deploys/66cc98d601565a0008c244eb
Deploy Preview https://deploy-preview-12027--apollo-client-docs.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

changeset-bot[bot] commented 1 month ago

🦋 Changeset detected

Latest commit: 59bc64bb9568a4d2a895ca9224dad656227ddd42

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

This PR includes changesets to release 1 package | Name | Type | | -------------- | ----- | | @apollo/client | 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