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
Remove all usages of MutliRule
in detekt code base
#5192
Comments
KtLintMultiRule seems to be the last multi rule in source set. I will take a stab at it in #5312 Update: |
This issue is stale because it has been open 90 days with no activity. Please comment or this will be closed in 7 days. |
This is still a think, as suggested by @chao2zhang. |
Have you started to work on it @chao2zhang? I could pick it up If you like. |
I have not. Please feel free to explore, although I think updating to ktlint 0.49 would be a prerequisite |
We decided to remove the
MultiRule
construct for all the reasons mentioned in #4240. As it is part of the API we cannot remove support for it just yet but we can change all those places where we use it for first party rules.This issue keeps track of all the tasks necessary.
EmptyBlocks
mutli rule (Run all rules from EmptyBlocks multi rule individually #5208)FileParsingRule
mutli rule (Remove multi rule FileParsingRule #5193)KDocStyle
mutli rule (Run all rules from KDocStyle multi rule individually #5207)KtLintMultiRule
(ReplaceKtLintMultiRule
#6094)NamingRules
multi rule (Remove rule from NamingRules multi rule #5212)KtLintMultiRule
#6094)MultiRule
(Delete MultiRule #6288)The text was updated successfully, but these errors were encountered: