fix(rule/variable-name): Use ruleArguments
property
#1565
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
PR Checklist
status: accepting prs
Hello again! 😄
Overview
As mentioned in the linked issue, one problem was that the arguments for
"@typescript-eslint/naming-convention"
weren't being passed to the ESLint rule properly, so it just said"error"
. From my understanding, this caused problems since this would also include lint errors for function names, etc. tooIt seems the fix was just a typo in the plugin. I also added a test specifically for
ban-keywords
, so it is a little more clear what is going on. This code generates the following ESLint file (for the sametslint.json
file in the linked issue):.eslintrc
Also it was stated in the linked issue that the best thing would possibly be "only outputing the
id-denylist
" rule - this doesn't address that part specifically, but does fix the bug within the rule.To make sure this issue doesn't exist for other plugins, maybe it might be a good idea to validate whatever a plugin returns against some schema, at least in testing? That way, a plugin won't accidentally set
rules
instead ofruleArguments
without some alarms going off.