Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update dependencies #782

Merged
merged 1 commit into from Mar 8, 2023
Merged

Update dependencies #782

merged 1 commit into from Mar 8, 2023

Conversation

lukellmann
Copy link
Member

The wrapper task is no longer configured to use the all distribution type. This results in smaller downloads (e.g. in CI) while still being able to inspect sources in IntelliJ (they are downloaded automatically).

Nothing is serializable since Kotlin 1.8.0 and kotlinx.serialization 1.5.0-RC which means dev.kord.common.entity.DiscordNull and dev.kord.gateway.NullDecoder are now obsolete.

Kotlin/dokka#2796 was fixed with Dokka 1.8.10, which means the dev build used since #755 is no longer needed and the corresponding maven repository was removed from the build files.

* Gradle 8.0 -> 8.0.2
* Kotlin 1.8.0 -> 1.8.10
* KSP 1.8.0-1.0.8 -> 1.8.10-1.0.9
* Ktor 2.2.1 -> 2.2.4
* kotlinx.serialization 1.4.1 -> 1.5.0
* Dokka 1.8.0-dev-194 -> 1.8.10
* AtomicFU 0.19.0 -> 0.20.0
* Binary compatibility validator 0.12.1 -> 0.13.0
* JUnit 5 5.9.1 -> 5.9.2
* MockK 1.13.2 -> 1.13.4

The wrapper task is no longer configured to use the all distribution
type. This results in smaller downloads (e.g. in CI) while still being
able to inspect sources in IntelliJ (they are downloaded automatically).

Nothing is serializable since Kotlin 1.8.0 and kotlinx.serialization
1.5.0-RC which means dev.kord.common.entity.DiscordNull and
dev.kord.gateway.NullDecoder are now obsolete.

Kotlin/dokka#2796 was fixed with Dokka 1.8.10,
which means the dev build used since #755 is no longer needed and the
corresponding maven repository was removed from the build files.
@lukellmann
Copy link
Member Author

@DRSchlaubi is there anything that would argue against using the bin Gradle distribution type instead of all?

@DRSchlaubi
Copy link
Member

If you use all IntelliJ doesn't need to download the src distribution type separately

@lukellmann
Copy link
Member Author

If you use all IntelliJ doesn't need to download the src distribution type separately

but if i saw this right, IntelliJ only downloads the src, while the all distribution also includes the whole html userguide, javadoc and dsl reference

@DRSchlaubi
Copy link
Member

Sounds logical, then it doesn't really make sense to use all ig

@lukellmann lukellmann merged commit 3bfe3ba into 0.8.x Mar 8, 2023
@lukellmann lukellmann deleted the dependencies branch March 8, 2023 06:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants