perf(prefer-importing-jest-globals): stop collecting import specifiers for no reason

[eryue0220]

This pr is to remove unused ImportDeclaration check.

[G-Rath]

Can you explain why this isn't needed? It must have been being covered by at least one test so it can't have been unused, though the fact that our suite still passes would suggest it's not needed but it would be good to understand why

[eryue0220]

Can you explain why this isn't needed? It must have been being covered by at least one test so it can't have been unused, though the fact that our suite still passes would suggest it's not needed but it would be good to understand why

I have two thoughts about that, the variables importedFunctionsWithSource only collect import elements from import statement, but what it really check is here, another is performance, if we don't use that, but it still iterates the AST nodes to collect something, it's a little worthless.

[G-Rath]

Oh I see, we're only ever writing to importedFunctionsWithSource - we never actually read from it; nice catch!

[github-actions]

🎉 This issue has been resolved in version 28.8.2 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀