pinterest/ktlint

Remove MavenDependencyResolver

bethcutler opened this issue · 4 comments

bethcutler commented

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.

shashachu commented

Makes total sense to me. Anyone who needs a custom reporter can run it from a local jar.

Tapchicoma commented

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.

  • 👍2
shashachu commented

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.

MaxAller commented

When is this going to be released? We're actively waiting on this change :)