-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
convert settings.gradle to kotlin #5147
Conversation
0a51642
to
4c75c5c
Compare
Seems straight forward enough, although I wonder is there any particular reasoning / added benefit to the change? I know sometimes Groovy stuff can be slow on its own. |
gradle switched the default: |
4c75c5c
to
d34bc1f
Compare
how do you normally get a review, a feedback or a merge, @Cervator @jdrueckert @BenjaminAmos ? |
Easiest way to get attention is via discord. We do have a review-hotlist channel there. I personally also use the activity-feed channel which is synced with some repos and automatically shows new PRs, commits and comments. Admittedly sometimes I see new changes but don't have time in that moment and then they slip my mind. Probably similar for others as well due to day job and other real life obligations. It also frequently happens that people comment to ask something but when the answer comes don't approve. Or approve but don't merge. Not entirely sure why that is though. |
@soloturn From my pov this can be merged once the checks are green again. |
convert settings.gradle to kotlin