From d114945b765d02269f69bd5cedff318dec067bb2 Mon Sep 17 00:00:00 2001 From: Ilya Goncharov Date: Thu, 20 Feb 2020 19:00:12 +0300 Subject: [PATCH] [Gradle, JS] Change message in polite manner #KT-36489 fixed #KT-36843 fixed --- .../jetbrains/kotlin/gradle/targets/js/KotlinJsPlugin.kt | 6 ++---- .../kotlin/gradle/targets/js/KotlinJsTargetPreset.kt | 7 +++---- .../kotlin/gradle/targets/js/ir/KotlinJsIrTargetPreset.kt | 7 +++---- 3 files changed, 8 insertions(+), 12 deletions(-) diff --git a/libraries/tools/kotlin-gradle-plugin/src/main/kotlin/org/jetbrains/kotlin/gradle/targets/js/KotlinJsPlugin.kt b/libraries/tools/kotlin-gradle-plugin/src/main/kotlin/org/jetbrains/kotlin/gradle/targets/js/KotlinJsPlugin.kt index bcef0fd0d61..567efdf4f45 100644 --- a/libraries/tools/kotlin-gradle-plugin/src/main/kotlin/org/jetbrains/kotlin/gradle/targets/js/KotlinJsPlugin.kt +++ b/libraries/tools/kotlin-gradle-plugin/src/main/kotlin/org/jetbrains/kotlin/gradle/targets/js/KotlinJsPlugin.kt @@ -44,12 +44,10 @@ open class KotlinJsPlugin( if (kotlinExtension._target == null) { project.logger.warn( """ - Need to initialize js target. - Use + Please initialize the Kotlin/JS target. Use: kotlin { js { - // Choose sub target (or both), for which js is necessary - // Affect in which tests are executed and final dist (in browser is only one bundle file) + // To build distributions for and run tests on browser or Node.js use one or both of: browser() nodejs() } diff --git a/libraries/tools/kotlin-gradle-plugin/src/main/kotlin/org/jetbrains/kotlin/gradle/targets/js/KotlinJsTargetPreset.kt b/libraries/tools/kotlin-gradle-plugin/src/main/kotlin/org/jetbrains/kotlin/gradle/targets/js/KotlinJsTargetPreset.kt index 2f42f2d636d..732eac94619 100644 --- a/libraries/tools/kotlin-gradle-plugin/src/main/kotlin/org/jetbrains/kotlin/gradle/targets/js/KotlinJsTargetPreset.kt +++ b/libraries/tools/kotlin-gradle-plugin/src/main/kotlin/org/jetbrains/kotlin/gradle/targets/js/KotlinJsTargetPreset.kt @@ -45,12 +45,11 @@ open class KotlinJsTargetPreset( if (!isBrowserConfigured && !isNodejsConfigured) { project.logger.warn( """ - Choose sub target (or both), for which js is necessary - In next releases it will be error - Use + Please choose a JavaScript environment to build distributions and run tests. + Not choosing any of them will be an error in the future releases. kotlin { js { - // Affect in which tests are executed and final dist (in browser is only one bundle file) + // To build distributions for and run tests on browser or Node.js use one or both of: browser() nodejs() } diff --git a/libraries/tools/kotlin-gradle-plugin/src/main/kotlin/org/jetbrains/kotlin/gradle/targets/js/ir/KotlinJsIrTargetPreset.kt b/libraries/tools/kotlin-gradle-plugin/src/main/kotlin/org/jetbrains/kotlin/gradle/targets/js/ir/KotlinJsIrTargetPreset.kt index b665a2097e2..4fa883f7588 100644 --- a/libraries/tools/kotlin-gradle-plugin/src/main/kotlin/org/jetbrains/kotlin/gradle/targets/js/ir/KotlinJsIrTargetPreset.kt +++ b/libraries/tools/kotlin-gradle-plugin/src/main/kotlin/org/jetbrains/kotlin/gradle/targets/js/ir/KotlinJsIrTargetPreset.kt @@ -31,12 +31,11 @@ open class KotlinJsIrTargetPreset( if (!isBrowserConfigured && !isNodejsConfigured) { project.logger.warn( """ - Choose sub target (or both), for which js is necessary - In next releases it will be error - Use + Please choose a JavaScript environment to build distributions and run tests. + Not choosing any of them will be an error in the future releases. kotlin { js { - // Affect in which tests are executed and final dist (in browser is only one bundle file) + // To build distributions for and run tests on browser or Node.js use one or both of: browser() nodejs() }