Skip to content

Commit

Permalink
Disable detekt because of the CVE (opensearch-project#497)
Browse files Browse the repository at this point in the history
Signed-off-by: bowenlan-amzn <[email protected]>

Signed-off-by: bowenlan-amzn <[email protected]>
  • Loading branch information
bowenlan-amzn authored Sep 8, 2022
1 parent 39be4e3 commit 50ac1e9
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 50ac1e9

Please sign in to comment.