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

Use libs.versions.toml and clean gradle files #719

Merged
merged 6 commits into from
Nov 17, 2022
Merged

Conversation

lukellmann
Copy link
Member

@lukellmann lukellmann commented Nov 12, 2022

Declaring the version catalog in libs.versions.toml instead of settings.gradle.kts has the advantage that it can be reused in buildSrc. It also introduces a single place to define dependencies and their version.

General cleanup of gradle files was also done, including:

Declaring the version catalog in libs.versions.toml instead of
settings.gradle.kts has the advantage that it can be reused in buildSrc.
It also introduces a single place to define dependencies and their
version.

General cleanup of gradle files was also done, including:

 * using default output for dokka: relying on defaults means less config
   and maintenance burden

 * removing DocsTask because it was unused, last usage of this custom
   task was removed in #367

 * getting rid of the source set workaround introduced in #386 by moving
   TweetNaclFast from voice/src/main/kotlin to voice/src/main/java
@lukellmann
Copy link
Member Author

@DRSchlaubi this will have to be updated to build/dokka/htmlMultiModule after this PR too (just like here)

build.gradle.kts Outdated Show resolved Hide resolved
.github/workflows/deployment-ci.yml Show resolved Hide resolved
build.gradle.kts Outdated Show resolved Hide resolved
lukellmann added a commit to lukellmann/kord-dokka that referenced this pull request Nov 16, 2022
@lukellmann lukellmann merged commit 338e882 into 0.8.x Nov 17, 2022
@lukellmann lukellmann deleted the version-catalog-toml branch November 17, 2022 12:38
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.

4 participants