-
Notifications
You must be signed in to change notification settings - Fork 512
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 MavenDependencyResolver #451
Comments
Makes total sense to me. Anyone who needs a custom reporter can run it from a local jar. |
I would propose to do it gracefully - for next release print warning message with a link to this issue, whenever 3rd party rule/reporter dependency is requested If nobody will provide solid arguments why it should stay - remove it in the release after. |
That's a great idea. |
When is this going to be released? We're actively waiting on this change :) |
See discussion in #445.
Basically, the Maven source seems to be a large dependency in ktlint for a small feature that may not be frequently used. I think we should consider removing this dependency.
The text was updated successfully, but these errors were encountered: