tidy up gradle plugins (mostly removing comments)

gradle-plugins/builder-plugins/build.gradle.kts

@@ -2,8 +2,6 @@ import org.jetbrains.kotlin.gradle.tasks.KotlinCompile
 
 plugins {
   id("kafkatorio.conventions.kotlin-dsl")
-//  `kotlin-dsl`
-//  kotlin("jvm") version embeddedKotlinVersion
 }
 
 dependencies {
@@ -43,21 +41,3 @@ tasks.withType<KotlinCompile>().configureEach {
     "-opt-in=kotlinx.serialization.ExperimentalSerializationApi",
   )
 }
-
-//val gradleJvmTarget = "11"
-//
-//kotlin {
-//  jvmToolchain {
-//    (this as JavaToolchainSpec).languageVersion.set(JavaLanguageVersion.of(gradleJvmTarget))
-//  }
-//}
-//
-//kotlinDslPluginOptions {
-//  jvmTarget.set(gradleJvmTarget)
-//}
-//
-//tasks.withType<KotlinCompile>().configureEach {
-//  kotlinOptions {
-//    jvmTarget = gradleJvmTarget
-//  }
-//}

gradle-plugins/factorio-gradle/build.gradle.kts

@@ -33,23 +33,6 @@ dependencies {
   implementation(libs.ktorSerialization.kotlinxJson)
 }
 
-
-//val gradleJvmTarget = 11
-//
-//kotlin {
-//  jvmToolchain(gradleJvmTarget)
-//}
-//
-//kotlinDslPluginOptions {
-//  jvmTarget.set("$gradleJvmTarget")
-//}
-//
-//tasks.withType<KotlinCompile>().configureEach {
-//  kotlinOptions {
-//    jvmTarget = "$gradleJvmTarget"
-//  }
-//}
-
 gradlePlugin {
   plugins {
     create("factorioMod") {
@@ -60,20 +43,7 @@ gradlePlugin {
   }
 }
 
-
-overrideKotlinLanguageVersion("1.7")
-
-//val gradleKotlinTarget = "1.7"
-//
-//afterEvaluate {
-//  tasks.withType<KotlinCompile>().configureEach {
-//    kotlinOptions {
-//      apiVersion = gradleKotlinTarget
-//      languageVersion = gradleKotlinTarget
-//    }
-//  }
-//}
-
+overrideKotlinLanguageVersion("1.7") // required for Kotlinx Serialization
 
 tasks.withType<KotlinCompile>().configureEach {
   kotlinOptions.freeCompilerArgs += listOf(

gradle-plugins/factorio-gradle/src/main/kotlin/dev/adamko/gradle/factorio/FactorioModPlugin.kt

@@ -14,14 +14,11 @@ abstract class FactorioModPlugin @Inject constructor(
   override fun apply(target: Project) {
     target.createSettings()
 
-
     val factorioModConfiguration =
       target.configurations.register(FACTORIO_GRADLE_CONFIGURATION_NAME) {
         asConsumer()
         factorioModAttributes(objects)
       }
-
-
   }
 
   private fun Project.createSettings(): FactorioModSettings {

gradle-plugins/settings-plugins/build.gradle.kts

@@ -8,6 +8,8 @@ plugins {
 
 val gradleJvmTarget = "11"
 
+description = "Provide Gradle settings plugins, used to configure base Gradle behaviour"
+
 dependencies {
   // This project should only be providing simple plugins for configuring Gradle without
   // dependencies. Try to avoid setting dependencies here, else the Gradle script classpath might

modules/web-map/build.gradle.kts

@@ -38,7 +38,6 @@ kotlin {
               "ws" to true,
             ),
           ),
-//          static = mutableListOf("$buildDir/processedResources/frontend/main")
           static = mutableListOf("$buildDir/processedResources/js/main")
         )
       }
@@ -139,7 +138,6 @@ val runWebMap by tasks.registering {
   group = rootProject.name
 
   dependsOn(tasks.withName("browserDevelopmentRun"))
-
 }