search

jestjs / jest

Delightful JavaScript Testing.
https://jestjs.io
MIT License
44.12k stars 6.44k forks source link

Cache interruptedKeys list in watch keypress event #15228

Open Connormiha opened 2 months ago

Connormiha commented 2 months ago

Summary

I cached interruptedKeys to avoid creating a new collection and calling getSortedUsageRows on each keypress.

Test plan

netlify[bot] commented 2 months ago

Deploy Preview for jestjs ready!

Built without sensitive environment variables

Name Link
Latest commit 7f731f4384b21898c4d6cce31240b5a74722dd4c
Latest deploy log https://app.netlify.com/sites/jestjs/deploys/66ab3722bad0260008383309
Deploy Preview https://deploy-preview-15228--jestjs.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.

SimenB commented 2 months ago

Huh, this somehow broke some tests

Connormiha commented 2 months ago

Huh, this somehow broke some tests

I will research it