Skip to content

fix(no-undefined-types): allow any available identifier; fixes #178,#1342 #1363

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Conversation

sebastian-fredriksson-bernholtz
Copy link
Contributor

Fixes #178 and #1342

@sebastian-fredriksson-bernholtz sebastian-fredriksson-bernholtz force-pushed the fix-allow-any-available-identifier-in-no-undefined-types branch from 44e5f6f to d24617e Compare March 16, 2025 11:22
realised that performance matter a lot for eslint, and functional map, recursive calls and converting forth and back between set and creating new arrays would not be very performant, so changed implementation to old style loops and working with a single SET
@brettz9
Copy link
Collaborator

brettz9 commented Mar 17, 2025

LGTM... Nice work!

@brettz9 brettz9 merged commit bf0acfe into gajus:main Mar 17, 2025
5 checks passed
Copy link

🎉 This issue has been resolved in version 50.6.7 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

no-undefined-types doesn't read local variables
2 participants