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
Chore: avoid handling rules instances in config-validator #9364
Conversation
LGTM |
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.
Nevermind, saw the link to the original issue.
The goal of this change is to progress towards being able to remove the |
Just to clarify, when you are talking about |
Yes, that's correct. |
@not-an-aardvark Could you rebase please? |
This is a modified version of #9277, which was merged but then reverted later for performance reasons. The performance issues from #9177 have been fixed (`Linter` no longer creates a new map of rules whenever it lints text). The goal of this change is to progress towards being able to remove the `rules` property from `Linter` instances. Unfortunately, this isn't possible yet, since there are a few other modules that rely on reading the list of defined rules.
f2f22a6
to
cdc6dbe
Compare
What is the purpose of this pull request? (put an "X" next to item)
[ ] Documentation update
[ ] 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
[x] Other, please explain:
What changes did you make? (Give an overview)
This is a modified version of #9277, which was merged but then reverted later for performance reasons. The performance issues from #9177 have been fixed (
Linter
no longer creates a new map of rules whenever it lints text).The goal of this change is to progress towards being able to remove the
rules
property fromLinter
instances. Unfortunately, this isn't possible yet, since there are a few other modules that rely on reading the list of defined rules.Is there anything you'd like reviewers to focus on?
Nothing in particular