chore: Remove deprecated APIs

Author: osipxd

CHANGELOG.md

@@ -32,6 +32,7 @@
 
 - Target JVM 1.8 → 17 
 - Gradle 7.2 → 8.13
+- Remove old deprecated APIs
 
 ## [0.10.1] - 2021-11-07
 

src/main/kotlin/BukkitExtension.kt

@@ -4,17 +4,13 @@ import org.gradle.api.Action
 import org.gradle.api.Project
 import org.gradle.api.model.ObjectFactory
 import org.gradle.api.provider.Property
-import org.gradle.kotlin.dsl.assign
 import org.gradle.kotlin.dsl.getByType
 import org.gradle.kotlin.dsl.property
-import org.slf4j.LoggerFactory
 import ru.endlesscode.bukkitgradle.extensions.finalizedOnRead
-import ru.endlesscode.bukkitgradle.extensions.warnSyntaxChanged
 import ru.endlesscode.bukkitgradle.plugin.extension.PluginConfigurationImpl
 import ru.endlesscode.bukkitgradle.server.ServerConstants
 import ru.endlesscode.bukkitgradle.server.extension.ServerConfigurationImpl
 
-// TODO 1.0: Remove deprecated fields on release
 public open class BukkitExtension internal constructor(
     public final override val plugin: PluginConfigurationImpl,
     public final override val server: ServerConfigurationImpl,
@@ -25,21 +21,6 @@ public open class BukkitExtension internal constructor(
         .convention(ServerConstants.DEFAULT_VERSION)
         .finalizedOnRead()
 
-    private val logger = LoggerFactory.getLogger("BukkitExtension")
-
-    @Deprecated("Use 'server' instead", ReplaceWith("server"))
-    public val run: ServerConfigurationImpl
-        get() {
-            logger.warnSyntaxChanged("bukkit.run", "bukkit.server")
-            return server
-        }
-
-    @Deprecated("Use 'server { ... }' instead", ReplaceWith("server(body)"))
-    public fun run(body: Action<ServerConfigurationImpl>) {
-        logger.warnSyntaxChanged("bukkit.run { ... }", "bukkit.server { ... }")
-        server(body)
-    }
-
     public fun server(body: Action<ServerConfigurationImpl>) {
         body.execute(server)
     }
@@ -53,12 +34,6 @@ public open class BukkitExtension internal constructor(
         plugin(body)
     }
 
-    @Deprecated("Use apiVersion instead of version.", ReplaceWith("apiVersion = version"))
-    public fun setVersion(version: String) {
-        logger.warnSyntaxChanged("bukkit.version = '...'", "bukkit.apiVersion = '...'")
-        apiVersion = version
-    }
-
     /** Disables plugin.yml generation. */
     @Deprecated(
         "Use 'plugin.disablePluginYamlGeneration()' instead",

src/main/kotlin/extensions/Logger.kt

@@ -5,11 +5,8 @@ import org.gradle.api.provider.ListProperty
 import org.gradle.api.provider.Property
 import org.gradle.kotlin.dsl.listProperty
 import org.gradle.kotlin.dsl.property
-import org.slf4j.LoggerFactory
 import ru.endlesscode.bukkitgradle.extensions.finalizedOnRead
-import ru.endlesscode.bukkitgradle.extensions.warnSyntaxChanged
 
-// TODO 1.0: Remove deprecated methods
 public class PluginConfigurationImpl(objects: ObjectFactory) : PluginConfiguration {
 
     private val _generatePluginYaml: Property<Boolean> = objects.property<Boolean>()
@@ -25,44 +22,6 @@ public class PluginConfigurationImpl(objects: ObjectFactory) : PluginConfigurati
     override val url: Property<String> = objects.property()
     override val authors: ListProperty<String> = objects.listProperty()
 
-    private val logger = LoggerFactory.getLogger("PluginConfiguration")
-
-    @Deprecated("Use property syntax instead", ReplaceWith("this.name.set(name)"))
-    public fun setName(name: String) {
-        logger.warnSyntaxChanged("bukkit.plugin.name = '...'", "bukkit.plugin.name.set('...')")
-        this.name.set(name)
-    }
-
-    @Deprecated("Use property syntax instead", ReplaceWith("this.description.set(description)"))
-    public fun setDescription(description: String) {
-        logger.warnSyntaxChanged("bukkit.plugin.description = '...'", "bukkit.plugin.description.set('...')")
-        this.description.set(description)
-    }
-
-    @Deprecated("Use property syntax instead", ReplaceWith("this.main.set(main)"))
-    public fun setMain(main: String) {
-        logger.warnSyntaxChanged("bukkit.plugin.main = '...'", "bukkit.plugin.main.set('...')")
-        this.main.set(main)
-    }
-
-    @Deprecated("Use property syntax instead", ReplaceWith("this.version.set(version)"))
-    public fun setVersion(version: String) {
-        logger.warnSyntaxChanged("bukkit.plugin.version = '...'", "bukkit.plugin.version.set('...')")
-        this.version.set(version)
-    }
-
-    @Deprecated("Use property syntax instead", ReplaceWith("this.url.set(url)"))
-    public fun setUrl(url: String) {
-        logger.warnSyntaxChanged("bukkit.plugin.url = '...'", "bukkit.plugin.url.set('...')")
-        this.url.set(url)
-    }
-
-    @Deprecated("Use property syntax instead", ReplaceWith("this.authors.set(authors)"))
-    public fun setAuthors(authors: List<String>) {
-        logger.warnSyntaxChanged("bukkit.plugin.authors = [...]", "bukkit.plugin.authors.set([...])")
-        this.authors.set(authors)
-    }
-
     /** Disables plugin.yaml parsing and generation. */
     public fun disablePluginYamlGeneration() {
         _generatePluginYaml.set(false)

src/main/kotlin/plugin/extension/PluginConfigurationImpl.kt

@@ -33,7 +33,7 @@ class PrepareServerSpec extends PluginSpecification {
         given: "configured online-mode"
         buildFile << """
             bukkit {
-                run {
+                server {
                     onlineMode = false
                 }
             }