-
Notifications
You must be signed in to change notification settings - Fork 186
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
Add linter_level
to Linter()
metadata.
#2351
Comments
~10% of the time For maximum backward compatibility, I suggest it to default to |
I've actually had in mind something along these lines, namely, that most linters could be toggled very easily between It would be nice to have an easy API through |
Working on this atm (diff will be large because it needs to touch all linters for the performance benefits) |
We can see 30% of the time is spent in
xml_find_all()
.What surprised me was
is_lint_level()
taking a whole 8s. That may be worth converting to an attribute ofLinter()
?Originally posted by @AshesITR in #2350 (comment)
The text was updated successfully, but these errors were encountered: