Skip to content

Commit

Permalink
Disable detekt because of the CVE (opensearch-project#497) (opensearc…
Browse files Browse the repository at this point in the history
…h-project#500)

Signed-off-by: bowenlan-amzn <bowenlan23@gmail.com>

Signed-off-by: bowenlan-amzn <bowenlan23@gmail.com>
(cherry picked from commit 50ac1e9)

Co-authored-by: bowenlan-amzn <bowenlan23@gmail.com>
Signed-off-by: Angie Zhang <langelzh@amazon.com>
  • Loading branch information
2 people authored and Angie Zhang committed Sep 12, 2022
1 parent 6bb9203 commit 8e23045
Showing 1 changed file with 7 additions and 6 deletions.
13 changes: 7 additions & 6 deletions build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -59,7 +59,8 @@ buildscript {
classpath "org.opensearch.gradle:build-tools:${opensearch_version}"
classpath "org.jetbrains.kotlin:kotlin-gradle-plugin:${kotlin_version}"
classpath "org.jetbrains.kotlin:kotlin-allopen:${kotlin_version}"
classpath "io.gitlab.arturbosch.detekt:detekt-gradle-plugin:1.21.0"
// TODO because snakeyaml CVE issue is on the latest version, have to disable detekt for now
// classpath "io.gitlab.arturbosch.detekt:detekt-gradle-plugin:1.21.0"
classpath "org.jacoco:org.jacoco.agent:0.8.7"
}
}
Expand All @@ -75,7 +76,7 @@ apply plugin: 'idea'
apply plugin: 'opensearch.opensearchplugin'
apply plugin: 'opensearch.testclusters'
apply plugin: 'opensearch.rest-test'
apply plugin: 'io.gitlab.arturbosch.detekt'
//apply plugin: 'io.gitlab.arturbosch.detekt'
apply plugin: 'org.jetbrains.kotlin.jvm'
apply plugin: 'org.jetbrains.kotlin.plugin.allopen'
apply plugin: 'opensearch.pluginzip'
Expand Down Expand Up @@ -147,10 +148,10 @@ task ktlintFormat(type: JavaExec, group: "formatting") {
args "-F", "src/**/*.kt", "spi/src/main/**/*.kt"
}

detekt {
config = files("detekt.yml")
buildUponDefaultConfig = true
}
//detekt {
// config = files("detekt.yml")
// buildUponDefaultConfig = true
//}

configurations.testImplementation {
exclude module: "securemock"
Expand Down

0 comments on commit 8e23045

Please sign in to comment.