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
Update: no-restricted-modules handle TemplateLiteral (fixes #12926) #12927
Conversation
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.
Thanks for the PR! I left a couple of notes.
10cc836
to
88d5d78
Compare
Also, the commit message should start with Alternatively, you can add more commits to this PR so we could just change the PR title. |
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, thanks!
Thanks for contributing to ESLint! |
) (eslint#12927) * Update: no-restricted-modules handle TemplateLiteral (fixes eslint#12926) * add fixture using backslashes * finally understood, I think
Prerequisites checklist
What is the purpose of this pull request? (put an "X" next to an item)
[ ] Documentation update
[x] Bug fix (template)
[ ] New rule (template)
[ ] Changes an existing rule (template)
[ ] Add autofixing to a rule
[ ] Add a CLI option
[ ] Add something to the core
[ ] Other, please explain:
Issue: #12926
What changes did you make? (Give an overview)
Added support for
TemplateLiteral
argument check on top of existing stringLiteral
check tono-restricted-modules
rule.Is there anything you'd like reviewers to focus on?