-
Notifications
You must be signed in to change notification settings - Fork 1k
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
[24.0] Make sure that all Linter subclasses are imported for listing them #18339
[24.0] Make sure that all Linter subclasses are imported for listing them #18339
Conversation
needed in planemo
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.
I think that's fine.
Given that we are touching this method here, and assuming the method name is a typo from #17081), should we change it back to |
You also need to fix the type in |
@nsoranzo Can't find a solution to define an alias for a class method (and in particular no nice solution). |
@bernt-matthias I'll push a fix to your branch. |
This PR was merged without a "kind/" label, please correct. |
Otherwise planemo prints out an error
Unknown linter type(s) ....
(and continues anyway ... but I think we should fix this).How to test the changes?
(Select all options that apply)
License