twoslashes / twoslash

Markup for generating rich type information in your documentations ahead of time.
https://twoslash.netlify.app/
MIT License
430 stars 16 forks source link

fix: check for undefined #22

Closed posva closed 5 months ago

posva commented 5 months ago

Hello, I don't have a reproduction for this. I encounter it only when deploying on Cloudflare. Looking at the code, it seems that this could be caught with TS with noUncheckedIndexedAccess.

Screenshot 2024-02-12 at 21 28 32

I didn't dig into the code, I just did a search of the property that was failing so the commit is very generic, sorry about that

netlify[bot] commented 5 months ago

Deploy Preview for twoslash ready!

Name Link
Latest commit 07865a5a7d997057c76875862e87dfcbc840be2e
Latest deploy log https://app.netlify.com/sites/twoslash/deploys/65ca80255d71ae0008913268
Deploy Preview https://deploy-preview-22--twoslash.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.