-
Notifications
You must be signed in to change notification settings - Fork 122
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
Move to gradle. #325
Comments
I don't see how this is related to #302 (which just seems to be a documentation issue), but maybe I have misunderstood what #302 is about?
Kotlin support for what? You can already use Gradle with Kotlin to depend on AnvilGUI. Whether AnvilGUI uses Maven or Gradle doesn't matter at all for the users of the library. In the end Maven and Gradle are compatible with each other when it comes to consuming dependencies. So the decision to use Gradle for AnvilGUI seems to mostly be a matter of personal taste unless there is some specific Gradle feature that we want to use. I personally don't oppose moving to Gradle, so I trust whatever decision is made by the maintainers. |
im not talking about that. i was talking about the #302 to make it easier to implement. |
Yeah I had just misunderstood what #302 was about, your argument makes sense now. It seems doable to build Kotlin code with Maven, but I would also not be surprised if Jetbrains has given the Gradle plugins for Kotlin more love than the Maven counterparts, so I trust your experience if you say that it's much easier in Gradle (I have not used Kotlin much, so I wouldn't know). |
Moving to Gradle seems like an implementation detail to me, for someone who uses AnvilGUI it won't matter whether we use Maven or Gradle. The simplicity of Maven is very appealing, so unless we have a strong reason to move to Gradle, I don't think it makes sense to switch. That matter can be discussed if and when such a reason comes along. |
This will make our life easier when implementing #302
maven is relatively hard to add kotlin support, so it would be much more easier to make it in a Gradle environment.
The text was updated successfully, but these errors were encountered: