Skip to content

Commit

Permalink
Bump kotlinpoet from 1.17.0 to 1.18.1 (#324)
Browse files Browse the repository at this point in the history
* Bump kotlinpoet from 1.17.0 to 1.18.1

Bumps `kotlinpoet` from 1.17.0 to 1.18.1.

Updates `com.squareup:kotlinpoet` from 1.17.0 to 1.18.1
- [Release notes](https://github.com/square/kotlinpoet/releases)
- [Changelog](https://github.com/square/kotlinpoet/blob/main/docs/changelog.md)
- [Commits](square/kotlinpoet@1.17.0...1.18.1)

Updates `com.squareup:kotlinpoet-metadata` from 1.17.0 to 1.18.1
- [Release notes](https://github.com/square/kotlinpoet/releases)
- [Changelog](https://github.com/square/kotlinpoet/blob/main/docs/changelog.md)
- [Commits](square/kotlinpoet@1.17.0...1.18.1)

Updates `com.squareup:kotlinpoet-ksp` from 1.17.0 to 1.18.1
- [Release notes](https://github.com/square/kotlinpoet/releases)
- [Changelog](https://github.com/square/kotlinpoet/blob/main/docs/changelog.md)
- [Commits](square/kotlinpoet@1.17.0...1.18.1)

---
updated-dependencies:
- dependency-name: com.squareup:kotlinpoet
  dependency-type: direct:production
  update-type: version-update:semver-minor
- dependency-name: com.squareup:kotlinpoet-metadata
  dependency-type: direct:production
  update-type: version-update:semver-minor
- dependency-name: com.squareup:kotlinpoet-ksp
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>

* Migrate to kotlin.metadata

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Nicklas Ansman <[email protected]>
  • Loading branch information
dependabot[bot] and ansman authored Jul 20, 2024
1 parent d0ea927 commit 52204f7
Show file tree
Hide file tree
Showing 15 changed files with 45 additions and 108 deletions.
6 changes: 0 additions & 6 deletions build.gradle.kts
Original file line number Diff line number Diff line change
Expand Up @@ -3,12 +3,6 @@ plugins {
alias(libs.plugins.kotlinx.binaryCompatibilityValidator)
}

buildscript {
dependencies {
classpath(libs.shadow)
}
}

apiValidation {
allprojects.filterNot { it.path == ":api" }.mapTo(ignoredProjects) { it.name }
}
27 changes: 2 additions & 25 deletions compiler/build.gradle.kts
Original file line number Diff line number Diff line change
@@ -1,20 +1,8 @@
plugins {
id("published-library")
kotlin("kapt")
id("com.github.johnrengelman.shadow")
}

tasks.compileKotlin {
compilerOptions {
freeCompilerArgs.addAll(
"-opt-in=com.squareup.kotlinpoet.metadata.KotlinPoetMetadataPreview",
)
}
}

val shade by configurations.named("compileShaded")

@Suppress("UnstableApiUsage")
dependencies {
implementation(projects.api)
implementation(libs.auto.service.api)
Expand All @@ -23,21 +11,10 @@ dependencies {
kapt(libs.incap.compiler)
implementation(libs.auto.common)
implementation(libs.kotlinpoet.core)
shade(libs.kotlinpoet.metadata) {
exclude("org.jetbrains.kotlin")
exclude("com.squareup", "kotlinpoet")
exclude("com.google.guava")
exclude("com.google.auto", "auto-common")
}
implementation(libs.kotlinpoet.metadata)
implementation(libs.kotlinpoet.ksp)
shade(libs.kotlinx.metadata) {
exclude("org.jetbrains.kotlin", "kotlin-stdlib")
}
implementation(libs.kotlin.metadata)
implementation(libs.oldestSupportedMoshi)
implementation(libs.ksp.api)
implementation(libs.asm)
}

tasks.shadowJar {
relocate("com.squareup.kotlinpoet.metadata", "se.ansman.kotshi.compiler.kotlinpoet.metadata")
}
Original file line number Diff line number Diff line change
Expand Up @@ -4,9 +4,9 @@ package se.ansman.kotshi.kapt

import com.google.auto.common.MoreElements
import com.google.common.collect.SetMultimap
import kotlinx.metadata.Modality
import kotlinx.metadata.isData
import kotlinx.metadata.modality
import kotlin.metadata.Modality
import kotlin.metadata.isData
import kotlin.metadata.modality
import se.ansman.kotshi.Errors
import se.ansman.kotshi.JsonDefaultValue
import se.ansman.kotshi.JsonSerializable
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -26,13 +26,13 @@ import com.squareup.kotlinpoet.U_SHORT_ARRAY
import com.squareup.kotlinpoet.WildcardTypeName
import com.squareup.kotlinpoet.asTypeName
import com.squareup.kotlinpoet.tags.TypeAliasTag
import kotlinx.metadata.KmClassifier
import kotlinx.metadata.KmFlexibleTypeUpperBound
import kotlinx.metadata.KmType
import kotlinx.metadata.KmTypeProjection
import kotlinx.metadata.KmVariance
import kotlinx.metadata.isNullable
import kotlinx.metadata.isSecondary
import kotlin.metadata.KmClassifier
import kotlin.metadata.KmFlexibleTypeUpperBound
import kotlin.metadata.KmType
import kotlin.metadata.KmTypeProjection
import kotlin.metadata.KmVariance
import kotlin.metadata.isNullable
import kotlin.metadata.isSecondary
import se.ansman.kotshi.model.AnnotationModel
import javax.lang.model.element.AnnotationMirror
import javax.lang.model.element.AnnotationValue
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,13 +8,13 @@ import com.google.common.collect.SetMultimap
import com.squareup.kotlinpoet.DelicateKotlinPoetApi
import com.squareup.kotlinpoet.asTypeName
import com.squareup.kotlinpoet.asTypeVariableName
import kotlinx.metadata.ClassKind
import kotlinx.metadata.Modality
import kotlinx.metadata.Visibility
import kotlinx.metadata.isInner
import kotlinx.metadata.kind
import kotlinx.metadata.modality
import kotlinx.metadata.visibility
import kotlin.metadata.ClassKind
import kotlin.metadata.Modality
import kotlin.metadata.Visibility
import kotlin.metadata.isInner
import kotlin.metadata.kind
import kotlin.metadata.modality
import kotlin.metadata.visibility
import se.ansman.kotshi.Errors
import se.ansman.kotshi.ExperimentalKotshiApi
import se.ansman.kotshi.KotshiJsonAdapterFactory
Expand Down
10 changes: 5 additions & 5 deletions compiler/src/main/kotlin/se/ansman/kotshi/kapt/KmClasses.kt
Original file line number Diff line number Diff line change
@@ -1,8 +1,8 @@
package se.ansman.kotshi.kapt

import kotlinx.metadata.KmClass
import kotlinx.metadata.kind
import kotlin.metadata.KmClass
import kotlin.metadata.kind

val KmClass.isClass get() = kind == kotlinx.metadata.ClassKind.CLASS
val KmClass.isObject get() = kind == kotlinx.metadata.ClassKind.OBJECT
val KmClass.isEnum get() = kind == kotlinx.metadata.ClassKind.ENUM_CLASS
val KmClass.isClass get() = kind == kotlin.metadata.ClassKind.CLASS
val KmClass.isObject get() = kind == kotlin.metadata.ClassKind.OBJECT
val KmClass.isEnum get() = kind == kotlin.metadata.ClassKind.ENUM_CLASS
Original file line number Diff line number Diff line change
Expand Up @@ -7,12 +7,12 @@ import com.squareup.kotlinpoet.asClassName
import com.squareup.kotlinpoet.metadata.specs.ClassInspector
import com.squareup.kotlinpoet.metadata.specs.toTypeSpec
import com.squareup.kotlinpoet.tag
import kotlinx.metadata.KmClass
import kotlinx.metadata.isLocalClassName
import kotlinx.metadata.jvm.KotlinClassMetadata
import se.ansman.kotshi.Errors.javaClassNotSupported
import javax.lang.model.element.Element
import javax.lang.model.element.TypeElement
import kotlin.metadata.KmClass
import kotlin.metadata.isLocalClassName
import kotlin.metadata.jvm.KotlinClassMetadata

class MetadataAccessor(private val classInspector: ClassInspector) {
private val metadataPerType = mutableMapOf<ClassName, Metadata?>()
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -7,10 +7,10 @@ import com.squareup.kotlinpoet.TypeName
import com.squareup.kotlinpoet.TypeSpec
import com.squareup.kotlinpoet.TypeVariableName
import com.squareup.kotlinpoet.tag
import kotlinx.metadata.KmClass
import kotlinx.metadata.Visibility
import kotlinx.metadata.isInner
import kotlinx.metadata.visibility
import kotlin.metadata.KmClass
import kotlin.metadata.Visibility
import kotlin.metadata.isInner
import kotlin.metadata.visibility
import se.ansman.kotshi.Errors
import se.ansman.kotshi.Polymorphic
import se.ansman.kotshi.PolymorphicLabel
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,10 +3,10 @@ package se.ansman.kotshi.kapt.generators
import com.squareup.kotlinpoet.KModifier
import com.squareup.kotlinpoet.ParameterSpec
import com.squareup.kotlinpoet.tag
import kotlinx.metadata.KmClass
import kotlinx.metadata.KmConstructor
import kotlinx.metadata.isData
import kotlinx.metadata.jvm.signature
import kotlin.metadata.KmClass
import kotlin.metadata.KmConstructor
import kotlin.metadata.isData
import kotlin.metadata.jvm.signature
import se.ansman.kotshi.Errors
import se.ansman.kotshi.Errors.privateDataClassProperty
import se.ansman.kotshi.JsonSerializable
Expand Down
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
package se.ansman.kotshi.kapt.generators

import com.squareup.kotlinpoet.TypeSpec
import kotlinx.metadata.KmClass
import kotlin.metadata.KmClass
import se.ansman.kotshi.Errors
import se.ansman.kotshi.Types.Kotshi.jsonDefaultValue
import se.ansman.kotshi.kapt.KaptProcessingError
Expand Down
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
package se.ansman.kotshi.kapt.generators

import kotlinx.metadata.KmClass
import kotlinx.metadata.isData
import kotlin.metadata.KmClass
import kotlin.metadata.isData
import se.ansman.kotshi.Errors
import se.ansman.kotshi.kapt.MetadataAccessor
import se.ansman.kotshi.kapt.isObject
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -4,9 +4,9 @@ import com.squareup.kotlinpoet.ClassName
import com.squareup.kotlinpoet.KModifier
import com.squareup.kotlinpoet.STAR
import com.squareup.kotlinpoet.tag
import kotlinx.metadata.KmClass
import kotlinx.metadata.Modality
import kotlinx.metadata.modality
import kotlin.metadata.KmClass
import kotlin.metadata.Modality
import kotlin.metadata.modality
import se.ansman.kotshi.Errors
import se.ansman.kotshi.Errors.defaultSealedValueIsGeneric
import se.ansman.kotshi.Errors.multipleJsonDefaultValueInSealedClass
Expand Down
1 change: 0 additions & 1 deletion gradle-plugin/build.gradle.kts
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,6 @@ dependencies {
implementation(files(libs.javaClass.superclass.protectionDomain.codeSource.location))
api(libs.kotlin.gradlePlugin)
api(libs.dokka.gradlePlugin)
api(libs.shadow)
implementation(gradleKotlinDsl())
}

Expand Down
32 changes: 1 addition & 31 deletions gradle-plugin/src/main/kotlin/published-library.gradle.kts
Original file line number Diff line number Diff line change
@@ -1,7 +1,4 @@

import com.github.jengelman.gradle.plugins.shadow.ShadowExtension
import com.github.jengelman.gradle.plugins.shadow.tasks.ShadowJar
import com.github.jengelman.gradle.plugins.shadow.transformers.ServiceFileTransformer
import org.gradle.jvm.tasks.Jar
import org.jetbrains.dokka.gradle.DokkaTask

Expand Down Expand Up @@ -81,11 +78,7 @@ val publication = with(the<PublishingExtension>()) {
}

publications.register<MavenPublication>("kotshi") {
if (pluginManager.hasPlugin("com.github.johnrengelman.shadow")) {
the<ShadowExtension>().component(this)
} else {
from(components["java"])
}
from(components["java"])
artifact(sourcesJar)
artifact(dokkaJavadocJar)

Expand Down Expand Up @@ -116,29 +109,6 @@ val publication = with(the<PublishingExtension>()) {
}
}

pluginManager.withPlugin("com.github.johnrengelman.shadow") {
val shade: Configuration = configurations.create("compileShaded")
configurations.named("compileOnly") {
extendsFrom(shade)
}
configurations.named("testRuntimeOnly") {
extendsFrom(shade)
}

val shadowJar = tasks.named<ShadowJar>("shadowJar") {
archiveClassifier.set("")
configurations = listOf(shade)
isEnableRelocation = true
relocationPrefix = "se.ansman.kotshi${project.path.replace(':', '.').replace('-', '_')}"
transformers.add(ServiceFileTransformer())
}

artifacts {
runtimeOnly(shadowJar)
archives(shadowJar)
}
}

if (providers.gradleProperty("signArtifacts").orNull?.toBooleanStrict() == true) {
configure<SigningExtension> {
useGpgCmd()
Expand Down
9 changes: 3 additions & 6 deletions gradle/libs.versions.toml
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
[versions]
kotlin = "2.0.0"
auto-service = "1.1.1"
kotlinpoet = "1.17.0"
kotlinpoet = "1.18.1"
compileTesting = "0.5.1"
incap = "1.0.0"
ksp = "2.0.0-1.0.23"
Expand All @@ -14,6 +14,7 @@ kotlinx-binaryCompatibilityValidator = "org.jetbrains.kotlinx.binary-compatibili

[libraries]
kotlin-gradlePlugin = { module = "org.jetbrains.kotlin:kotlin-gradle-plugin", version.ref = "kotlin" }
kotlin-metadata = { module = "org.jetbrains.kotlin:kotlin-metadata-jvm", version.ref = "kotlin" }

dokka-gradlePlugin = "org.jetbrains.dokka:dokka-gradle-plugin:1.9.20"

Expand All @@ -26,8 +27,6 @@ kotlinpoet-core = { module = "com.squareup:kotlinpoet", version.ref = "kotlinpoe
kotlinpoet-metadata = { module = "com.squareup:kotlinpoet-metadata", version.ref = "kotlinpoet" }
kotlinpoet-ksp = { module = "com.squareup:kotlinpoet-ksp", version.ref = "kotlinpoet" }

kotlinx-metadata = "org.jetbrains.kotlinx:kotlinx-metadata-jvm:0.9.0"

junit-jupiter = "org.junit.jupiter:junit-jupiter:5.10.3"

assertk = "com.willowtreeapps.assertk:assertk-jvm:0.28.1"
Expand All @@ -47,6 +46,4 @@ ksp-api = { module = "com.google.devtools.ksp:symbol-processing-api", version.re
ksp-commonDeps = { module = "com.google.devtools.ksp:symbol-processing-common-deps", version.ref = "ksp" }
ksp-aaEmbeddable = { module = "com.google.devtools.ksp:symbol-processing-aa-embeddable", version.ref = "ksp" }

asm = "org.ow2.asm:asm:9.7"

shadow = "com.github.johnrengelman:shadow:8.1.1"
asm = "org.ow2.asm:asm:9.7"

0 comments on commit 52204f7

Please sign in to comment.