graphql-js
graphql-js copied to clipboard
graphql
Use a type-safe way is differentiating ExecutionResult from Array<Error>
Motivation
Checking a string key on an array is not valid, correctly yielding a warning on a TS branch I'm playing in. Narrow the type with Array.isArray instead.
Performance
Running yarn benchmark (which is amazing, btw) on my 2019 MacBook Pro running macOS 12.4 and node 18.2.0, I see a few slight increases and slight decreases of my local vs HEAD. I exited all other user processes on the system, but there was still some slight variability from run to run regardless.
Here's a screenshot of the run, let me know if I need to dive into CPU profiler data to find another type-safe alternative that's cheaper at runtime.
The committers listed above are authorized under a signed CLA.
- :white_check_mark: login: matthargett / name: Matt Hargett (b76e38cd434431aa2e8b72f607fec8ac25af37d5)
![linux-foundation-easycla[bot] avatar](https://avatars.githubusercontent.com/in/17893?v=4 "Published by a pub.dev verified publisher"){kind=small}
Deploy Preview for compassionate-pike-271cb3 ready!
| Name | Link |
|---|---|
| Latest commit | f488b12f65c8f927331b4fb2181356989e2cc4a3 |
| Latest deploy log | https://app.netlify.com/sites/compassionate-pike-271cb3/deploys/62ac0db1352f460008f7597e |
| Deploy Preview | https://deploy-preview-3649--compassionate-pike-271cb3.netlify.app |
| Preview on mobile | Toggle QR Code...Use your smartphone camera to open QR code link. |
To edit notification comments on pull requests, go to your Netlify site settings.
![netlify[bot] avatar](https://avatars.githubusercontent.com/in/13473?v=4 "Published by a pub.dev verified publisher"){kind=small}
Checks failing because lockfile does not match, our ci requires you to update this yourself and then we check to make sure it matches.
Putting together your typescript version changes and your proposed code change, it sounds like you are saying that on newest version of typescript, we get a warning on our explicit property check?
It could be the simplest thing is to just disable the warning for that line.
Could be also that this is not secondary to the version change and just a different tsconfig in your project? I skimmed the TS 4.7 release notes and didn't note anything at first glance.
![github-actions[bot] avatar](https://avatars.githubusercontent.com/in/15368?v=4 "Published by a pub.dev verified publisher"){kind=small}