-
-
Notifications
You must be signed in to change notification settings - Fork 929
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
Refactor to remove unneeded isLessVariable()
utility
#6995
Conversation
|
No grep match: ```console $ git grep 'isLessVariable' || echo '(no match)' (no match) ``` Also, this change adds a tiny script to find unused module files: ```console $ node scripts/find-unused-modules.mjs lib/utils/isStandardSyntaxMathFunction.js lib/utils/isStandardSyntaxMediaFeature.js ``` I will address other unused files later.
beadaa8
to
6099ef6
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thank you!
Once the 15.0.0 unneeded files are removed, let's rebase #6979 and run the script as part of that pull request as I suspect more files we made unneeded.
Okay, I'll do it. 👍🏼 |
No grep match: ```console $ git grep 'isLessVariable' || echo '(no match)' (no match) ``` Also, this change adds a tiny script to find unused module files: ```console $ node scripts/find-unused-modules.mjs lib/utils/isStandardSyntaxMathFunction.js lib/utils/isStandardSyntaxMediaFeature.js ``` I will address other unused files later.
This script file was added in #6995. We don't need it anymore since unused modules were resolved.
Remove unused `scripts/find-unused-modules.mjs` This script file was added in #6995. We don't need it anymore since unused modules were resolved.
None.
No grep match:
Also, this change adds a tiny script to find unused module files:
I will address other unused files
laterwith follow-up PRs.Note: See also #6971 (comment)
I tried the ESLint rule
import/no-unused-modules
, but it didn't work expectedly. Please help if anyone knows. If the rule works, it's preferred over the custom script.