[Suggestion] Migrate gradle files from Groovy to Kotlin DSL

[oas004]

Hey!

First of all, thanks for a great library!

This is just a suggestion and I wanted to know what you guys thought about it. Feel free to close it, if it does not make sense.

I noticed that all the gradle files are in Groovy, what do you guys think about converting it to Kotlin DSL? Is there a reason for not migrating or any downside to migrating? :)

I would love to help out and contribute if this is okay :)

One suggested approach to this as we did when migrating in the horologist project is the following:

1. Go module by module prepare the Groovy scripts following the steps described here. Then make a PR for each module to keep it easy to review.
2. Do the actual conversion of the file to .kts and check that it compiles and that everything works as expected. Then make a PR for each module, to make it easy to review.

One tool that might help migrate is this script to convert Groovy to Kotlin DSL. I have not tried it myself though.

[bentrengrove]

Thanks for the suggestion. We would definitely accept this as a contribution if you are open to doing it!

I agree with your plan to do it module by module.

[oas004]

I would love to! :) I will try to prioritize this :)

[github-actions]

This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 5 days.

[bentrengrove]

I guess stalebot can't see all the PRs

[oas004]

Just to give a status update here is that currently 22/27 groovy files are now in Kotlin DSL :) We have PRs up on the rest except the project gradle file. I will do that when everything else is in. When that is done my thoughts where to set the target and source java versions to 11 and to update the plugins to using version catalog correctly :) Then I think we can also bump the AGP version as of #1617

[bentrengrove]

Thank you for all your effort!

[github-actions]

This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 5 days.