fix: FlutterFire CLI is not able to handle projects that use…

[devNamanG]

… Kotlin DSL for Gradle #305

Description

Added support for updating android/build.gradle.kts, android/settings.gradle.kts and android/app/build.gradle.kts files. Added RegExps, functions to handle Kotlin Gradle DSL files if Groovy Gradle DSL files are not found We assume that the project does not use a mix of Groovy and Kotlin DSL gradle files. We check whether the android/build.gradle file exists or not. If it exists, the existing code handles updating the files. If it does not not exist, we proceed to check whether android/build.gradle.kts file exists. If it does, we use Kotlin DSL configuration. If both configuration files are not found, an Exception is thrown.

Type of Change

• [ ] ✨ feat -- New feature (non-breaking change which adds functionality)
• [x] 🛠️ fix -- Bug fix (non-breaking change which fixes an issue)
• [ ] ❌ ! -- Breaking change (fix or feature that would cause existing functionality to change)
• [x] 🧹 refactor -- Code refactor
• [ ] ✅ ci -- Build configuration change
• [ ] 📝 docs -- Documentation
• [ ] 🗑️ chore -- Chore

[CLAassistant]

All committers have signed the CLA.

[devNamanG]

Okay so I tried running the code on a project with Kotlin DSL files, and for some reason, it is not updating the those files, even though it does write google-services.json. Can someone check this out as well? The existing plugin code seems to not even call the updated gradleContentUpdates function. It would be really helpful if a maintainer checks this out, thank you!

[github-actions[bot]]

Hello 👋, this PR has been opened for more than 2 months with no activity on it.

If you think this is a mistake please comment and ping a maintainer to get this merged ASAP! Thanks for contributing!

You have 15 days until this gets closed automatically

[Henry5775]

Has this been implemented? Still having this issue with my app even with latest dev build.

[devNamanG]

Hey @Henry5775 , no i don't think so, last i checked this was still not working. Then I just abandoned using Kotlin DSL and switched to Groovy DSL instead.

[jasecoulls]

Has this been implemented? Still having this issue with my app even with latest dev build.

Same here. Wasted so much time trying to find a solution to this.

[IldySilva]

do we have any update on that one? it's a must,i guess @russellwheatley

[alimaherofficial]

Is there anything new? we need these modifications!!

[Ehesp]

Let me bring this up internally 👍

[the-best-is-best]

this not support flutter 3.29 now flutter use kotlin DSL

[russellwheatley]

Tests will probably fail as there's a token required which isn't available for forked PRs, I ran locally and they all passed