-
Notifications
You must be signed in to change notification settings - Fork 6
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
introduce a BOM #222
introduce a BOM #222
Conversation
@@ -363,16 +363,19 @@ dependencies { | |||
// core coroutines | |||
implementation("org.jetbrains.kotlinx:kotlinx-coroutines-core:1.4.3") | |||
implementation("org.jetbrains.kotlinx:kotlinx-coroutines-android:1.4.3") | |||
|
|||
// a BOM ensures that all artifacts used from the library are of the same version | |||
implementation(platform("com.rickbusarow.dispatch:dispatch-bom:1.0.0-RC01")) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nit: You might want to tell the people that they have to add the platform tag to other configurations as well, i.e. testImplementation
and androidTestImplementation
. This alone is of course not very useful and saves no writing, however, people can set up their own "production" or "test" Java platform, import your BOM there and reference this here, so all their production / test dependencies are refererenced in a unified matter.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh wait, testImplementation
actually inherits implementation
, ignore me... I was thinking about kapt
and others for a moment. All good on your side :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
:) Thanks. I plan to update all of this documentation before the 1.0 release. I know it's pretty lacking.
* Bump com.diffplug.spotless from 5.12.4 to 5.12.5 (#220) * introduce a BOM (#222) * introduce a BOM * update main project README * Revert "introduce a BOM (#222)" (#224) This reverts commit ddf205a. * Bump io.gitlab.arturbosch.detekt from 1.16.0 to 1.17.0 (#226) Bumps io.gitlab.arturbosch.detekt from 1.16.0 to 1.17.0. Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump io.gitlab.arturbosch.detekt from 1.17.0 to 1.17.1 (#228) Bumps io.gitlab.arturbosch.detekt from 1.17.0 to 1.17.1. Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * dependency updates - detekt 1.17.1 - JUnit 5.7.2 - Kotest 4.6.0 - Kotlin 1.5.10 - Androidx AppCompat 1.3.0 - Androidx core-ktx 1.5.0 - Androidx Fragment 1.3.4 Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* dependency updates (#221) * dependency updates - AGP 4.2.1 - coroutines 1.5.0 - spotless 5.12.5 * update artifact versions in documentation samples Signed-off-by: GitHub Actions <actions@github.com> * Dispatch bom (#225) * Bump com.diffplug.spotless from 5.12.4 to 5.12.5 (#220) * introduce a BOM * update main project README Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * dependency updates (#229) * Bump com.diffplug.spotless from 5.12.4 to 5.12.5 (#220) * introduce a BOM (#222) * introduce a BOM * update main project README * Revert "introduce a BOM (#222)" (#224) This reverts commit ddf205a. * Bump io.gitlab.arturbosch.detekt from 1.16.0 to 1.17.0 (#226) Bumps io.gitlab.arturbosch.detekt from 1.16.0 to 1.17.0. Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump io.gitlab.arturbosch.detekt from 1.17.0 to 1.17.1 (#228) Bumps io.gitlab.arturbosch.detekt from 1.17.0 to 1.17.1. Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * dependency updates - detekt 1.17.1 - JUnit 5.7.2 - Kotest 4.6.0 - Kotlin 1.5.10 - Androidx AppCompat 1.3.0 - Androidx core-ktx 1.5.0 - Androidx Fragment 1.3.4 Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * remove JUnit from the dispatch-test classpath (#232) fixes #231 * stop generating a BuildConfig in the Android modules (#234) fixes #233 * fix coroutineScope cancellation with withViewLifecycle (#237) * change version to 1.0.0-beta10 * update CHANGELOG * change library version in docs from RC01 to beta10 * add use of the BOM to junit readmes Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
fixes #219