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

Migrating to Kotlinter gradle plugin. #184

Merged
merged 15 commits into from Mar 7, 2024

Conversation

AdamMc331
Copy link
Owner

Summary

Migrating off of previous plugin.

How It Was Tested

CI passing.

@github-actions
Copy link

github-actions bot commented Mar 9, 2023

1 Warning
⚠️ Please consider breaking up this pull request.
2 Messages
📖 Thanks @AdamMc331!
📖

The following dependencies have later milestone versions:

  • org.jacoco:org.jacoco.agent [0.8.8 -> 0.8.9]
    http://jacoco.org

Failed to determine the latest version for the following dependencies (use –info for details):

  • androidx.activity:activity-compose
  • androidx.appcompat:appcompat
  • androidx.compose:compose-bom
  • androidx.compose.compiler:compiler
  • androidx.compose.foundation:foundation
  • androidx.compose.material3:material3
  • androidx.compose.material3:material3-window-size-class
  • androidx.compose.ui:ui
  • androidx.compose.ui:ui-test-junit4
  • androidx.compose.ui:ui-test-manifest
  • androidx.compose.ui:ui-tooling
  • androidx.compose.ui:ui-tooling-preview
  • androidx.core:core-ktx
  • androidx.datastore:datastore
  • androidx.hilt:hilt-navigation-compose
  • androidx.lifecycle:lifecycle-runtime-ktx
  • androidx.navigation:navigation-compose
  • androidx.room:room-compiler
  • androidx.room:room-ktx
  • androidx.room:room-runtime
  • androidx.test.espresso:espresso-core
  • androidx.test.ext:junit
  • androidx.window:window
  • app.cash.turbine:turbine
  • com.android.tools:desugar_jdk_libs
  • com.google.accompanist:accompanist-navigation-animation
  • com.google.accompanist:accompanist-systemuicontroller
  • com.google.android.material:material
  • com.google.dagger:hilt-android
  • com.google.dagger:hilt-android-compiler
  • com.google.dagger:hilt-android-testing
  • com.google.dagger:hilt-compiler
  • com.google.devtools.ksp:symbol-processing
  • com.google.devtools.ksp:symbol-processing-api
  • com.google.devtools.ksp:symbol-processing-cmdline
  • com.google.protobuf:com.google.protobuf.gradle.plugin
  • com.google.protobuf:protobuf-javalite
  • com.google.protobuf:protoc
  • com.google.truth:truth
  • io.github.raamcosta.compose-destinations:animations-core
  • io.github.raamcosta.compose-destinations:ksp
  • io.mockk:mockk
  • junit:junit
  • org.jetbrains.kotlin:kotlin-annotation-processing-gradle
  • org.jetbrains.kotlin:kotlin-stdlib
    1.9.21
  • org.jetbrains.kotlinx:kotlinx-coroutines-test
    1.8.0

Gradle current updates:

  • Gradle: [8.6: UP-TO-DATE]

Generated by 🚫 Danger

@coveralls
Copy link

coveralls commented Apr 19, 2023

Coverage Status

Coverage: 67.104%. Remained the same when pulling c482bd9 on TOA-180/kotlinter_plugin into b0e7a57 on development.

build.gradle.kts Outdated
Comment on lines 10 to 17
classpath("com.android.tools.build:gradle:7.2.1")
classpath("org.jetbrains.kotlin:kotlin-gradle-plugin:1.7.21")
classpath("io.gitlab.arturbosch.detekt:detekt-gradle-plugin:1.19.0")
classpath("com.google.dagger:hilt-android-gradle-plugin:2.45")
classpath("com.github.ben-manes:gradle-versions-plugin:0.29.0")
classpath("org.jetbrains.kotlinx:kover:0.4.1")
classpath("com.hiya:jacoco-android:0.2")
classpath("gradle.plugin.org.kt3k.gradle.plugin:coveralls-gradle-plugin:2.12.0")
Copy link
Owner Author

Choose a reason for hiding this comment

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

Before we merge, let's move these into our toml file and define them properly, but we can continue to use hardcoded values here to get this compiling again.

@AdamMc331 AdamMc331 merged commit f48b964 into development Mar 7, 2024
4 checks passed
@AdamMc331 AdamMc331 deleted the TOA-180/kotlinter_plugin branch March 7, 2024 03:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants