it just takes a bit of time for idea to pick it up and remove the errors it shows

and i could.. merge the files into one as part of the PR.. should I ?

🤔

i don’t know bro

it looks a bit much

i prefer what you have right now

not going too far like the extension property thingy

good, its cool but its kind of hard to understand without having a IDE open

and the plugin versions are still basically all in one place

also you can replace

apply

at the top of the file with

plugins { id("someplugin") }

without a version specified

yeah but without version specified which version it will be used though?

the one loaded in the rootProject

or if its a builtin plugin its already loaded by gradle

thats why you do not specify a plugin version for jacoco , idea or java

also to simplify the buildscript even more one could vote on that: https://youtrack.jetbrains.com/issue/KT-27612

maybe a good / better way to structure the Versions and Plugins and Dependencies would be to have one object for each dependency / plugin

object serialization {
    const val id = "kotlinx-serialization"
    const val version = "0.8.0-rc13"
    const val dependency = "org.jetbrains.kotlinx:kotlinx-serialization-runtime:$version"
    const val module = "org.jetbrains.kotlin:kotlin-serialization"
}

one could add whatever properties required, most of the time its gonna be less properties for just compile dependency

but if you need more its less of mix and match between constants distributed over 3 files

named

serialization

or

kotlinxSerialization

or

serializationVersion

do you think its a good approach @kittinunf?

i like it!! @Nikky

i tried it in my own project.. turns into this: https://github.com/elytra/Voodoo/blob/master/buildSrc/src/main/kotlin/Constants.kt

could be part of #465 Fuel has less dependencies and modules than that mess i built, so it will be less effort to convert

but first updating my app to use the new release

@kittinunf for the wrapper thing, and to make sure it does not happen again:

tasks.withType<Wrapper> {
    gradleVersion = Gradle.version
    distributionType = Gradle.distributionType // Wrapper.DistributionType.ALL
}

needs to be at the very bottom of the buildscript apparently

ah nice …

i think this part of gradle is so annoying

still not sure if this will play nicely when there is major version change for gradle.. but i can hope

setup can be friction

sometimes i feel like i want something less magic-y less complicated than gradle …