Skip to content

Commit

Permalink
some refactoring
Browse files Browse the repository at this point in the history
  • Loading branch information
DatL4g committed Sep 1, 2024
1 parent 607eccd commit 5e229f7
Show file tree
Hide file tree
Showing 6 changed files with 10 additions and 8 deletions.
2 changes: 1 addition & 1 deletion jcef/build.gradle.kts
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ dependencies {
mavenPublishing {
publishToMavenCentral(host = SonatypeHost.S01, automaticRelease = true)
signAllPublications()
coordinates("dev.datlag", "jcef", "2024.04.20.1")
coordinates("dev.datlag", "jcef", "2024.04.20.2")

pom {
name.set(project.name)
Expand Down
2 changes: 1 addition & 1 deletion kcef/build.gradle.kts
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ tasks.dokkaHtmlPartial {
mavenPublishing {
publishToMavenCentral(host = SonatypeHost.S01, automaticRelease = true)
signAllPublications()
coordinates("dev.datlag", "kcef", "2024.04.20.1")
coordinates("dev.datlag", "kcef", "2024.04.20.2")

pom {
name.set(project.name)
Expand Down
1 change: 0 additions & 1 deletion kcef/src/main/kotlin/dev/datlag/kcef/KCEF.kt
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,6 @@ import dev.datlag.kcef.KCEF.InitRestartRequired
import dev.datlag.kcef.KCEF.NewClientOrNullError
import dev.datlag.kcef.common.existsSafely
import dev.datlag.kcef.common.suspendCatching
import dev.datlag.kcef.model.KCEFAcknowledge
import kotlinx.coroutines.flow.MutableStateFlow
import kotlinx.coroutines.flow.first
import kotlinx.coroutines.flow.update
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
package dev.datlag.kcef.model
package dev.datlag.kcef

@RequiresOptIn(message = "This API may be library and application breaking. Only use it if you fully understand what you are doing.")
@Retention(AnnotationRetention.BINARY)
Expand Down
7 changes: 5 additions & 2 deletions kcef/src/main/kotlin/dev/datlag/kcef/KCEFBuilder.kt
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,6 @@ import dev.datlag.kcef.KCEFBuilder.InitProgress.Builder.ProgressCallback
import dev.datlag.kcef.KCEFBuilder.Settings
import dev.datlag.kcef.common.*
import dev.datlag.kcef.model.GitHubRelease
import dev.datlag.kcef.model.KCEFAcknowledge
import dev.datlag.kcef.step.extract.TarGzExtractor
import dev.datlag.kcef.step.fetch.PackageDownloader
import dev.datlag.kcef.step.init.CefInitializer
Expand Down Expand Up @@ -680,7 +679,11 @@ class KCEFBuilder {
release.assets.filter { asset ->
platform.os.values.any { os ->
asset.name.contains(os, true) || asset.downloadUrl.contains(os, true)
}
} && asset.downloadUrl.isNotBlank()
}.filter { asset ->
platform.arch.values.any { arch ->
asset.name.contains(arch, ignoreCase = true) || asset.downloadUrl.contains(arch, true)
} && asset.downloadUrl.isNotBlank()
}.map { it.downloadUrl }
}
val platformPackageList = osPackageList.filter { url ->
Expand Down
4 changes: 2 additions & 2 deletions kcef/src/main/kotlin/dev/datlag/kcef/model/GitHubRelease.kt
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ internal data class GitHubRelease(
) {
@Serializable
internal data class Asset(
val name: String,
@SerialName("browser_download_url") val downloadUrl: String
val name: String = "",
@SerialName("browser_download_url") val downloadUrl: String = ""
)
}

0 comments on commit 5e229f7

Please sign in to comment.