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

updating testing and small increment dependencies #225

Merged
merged 3 commits into from
Sep 8, 2021

Conversation

lgleasain
Copy link
Contributor

this updates most of the testing dependencies along with a couple of other small incremental bumps.

@carrotkite
Copy link

JaCoCo Code Coverage 77.75% ✅

Class Covered Meta Status

Generated by 🚫 Danger

build.gradle Outdated
@@ -25,8 +25,8 @@ buildscript {
classpath "org.jetbrains.kotlin:kotlin-gradle-plugin:$kotlinVersion"
classpath "com.vanniktech:gradle-android-junit-jacoco-plugin:0.16.0"
classpath "com.github.ben-manes:gradle-versions-plugin:0.36.0"
classpath "org.jetbrains.dokka:dokka-gradle-plugin:1.4.32"
classpath "org.jetbrains.dokka:android-documentation-plugin:1.4.32"
classpath "org.jetbrains.dokka:dokka-gradle-plugin:1.5.0"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I wonder if this dokka update doesn't break publishing (it's usually pretty fragile integration). Not sure how to best test without trying to run the publish task. Maybe can try to publish to local maven.

Copy link
Contributor Author

@lgleasain lgleasain Sep 2, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah, ok. I'm going to hold off on the dokka update then for the time being and keep the rest rather than going down that hole right now.

@Laimiux Laimiux merged commit 9791680 into master Sep 8, 2021
@Laimiux Laimiux deleted the lance/updating_testing_and_point_dependencies branch September 8, 2021 21:06
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

4 participants