[Gradle] Finalize naming of 'runProjectConfigurationHealthCheckWhenEvaluated' API

^KT-48823
This commit is contained in:
sebastian.sellmair
2021-09-28 08:59:41 +02:00
committed by Space
parent 5430b19262
commit 4eaabe54ee
7 changed files with 11 additions and 11 deletions
@@ -227,7 +227,7 @@ class KotlinMultiplatformPlugin : Plugin<Project> {
UnusedSourceSetsChecker.checkSourceSets(project)
project.runPostEvaluationProjectConfigurationHealthCheck {
project.runProjectConfigurationHealthCheckWhenEvaluated {
checkSourceSetVisibilityRequirements(project)
}
}
@@ -20,7 +20,7 @@ import org.jetbrains.kotlin.gradle.plugin.sources.FragmentConsistencyChecker
import org.jetbrains.kotlin.gradle.plugin.sources.FragmentConsistencyChecks
import org.jetbrains.kotlin.gradle.utils.addExtendsFromRelation
import org.jetbrains.kotlin.gradle.utils.lowerCamelCaseName
import org.jetbrains.kotlin.gradle.utils.runPostEvaluationProjectConfigurationHealthCheck
import org.jetbrains.kotlin.gradle.utils.runProjectConfigurationHealthCheckWhenEvaluated
import org.jetbrains.kotlin.project.model.KotlinModuleDependency
import org.jetbrains.kotlin.project.model.KotlinModuleFragment
import org.jetbrains.kotlin.project.model.refinesClosure
@@ -56,7 +56,7 @@ open class KotlinGradleFragmentInternal @Inject constructor(
project.addExtendsFromRelation(getConfiguration(this), getConfiguration(other.get())) // todo eager instantiation; fix?
}
project.runPostEvaluationProjectConfigurationHealthCheck {
project.runProjectConfigurationHealthCheckWhenEvaluated {
kotlinGradleFragmentConsistencyChecker.runAllChecks(this@KotlinGradleFragmentInternal, other.get())
}
}
@@ -92,7 +92,7 @@ class DefaultKotlinSourceSet(
// Fail-fast approach: check on each new added edge and report a circular dependency at once when the edge is added.
checkForCircularDependencies()
project.runPostEvaluationProjectConfigurationHealthCheck {
project.runProjectConfigurationHealthCheckWhenEvaluated {
defaultSourceSetLanguageSettingsChecker.runAllChecks(this@DefaultKotlinSourceSet, other)
}
}
@@ -46,7 +46,7 @@ open class KotlinJsPlugin(
defaultJsCompilerType = PropertiesProvider(project).jsCompiler
}
project.runPostEvaluationProjectConfigurationHealthCheck {
project.runProjectConfigurationHealthCheckWhenEvaluated {
if (kotlinExtension._target == null) {
project.logger.warn(
"""
@@ -15,7 +15,7 @@ import org.jetbrains.kotlin.gradle.targets.js.KotlinJsTarget
import org.jetbrains.kotlin.gradle.targets.js.KotlinJsTargetConfigurator
import org.jetbrains.kotlin.gradle.targets.js.ir.KotlinJsIrTargetPreset
import org.jetbrains.kotlin.gradle.utils.lowerCamelCaseName
import org.jetbrains.kotlin.gradle.utils.runPostEvaluationProjectConfigurationHealthCheck
import org.jetbrains.kotlin.gradle.utils.runProjectConfigurationHealthCheckWhenEvaluated
import org.jetbrains.kotlin.statistics.metrics.StringMetrics
open class KotlinJsTargetPreset(
@@ -55,7 +55,7 @@ open class KotlinJsTargetPreset(
}
this.isMpp = this@KotlinJsTargetPreset.isMpp
project.runPostEvaluationProjectConfigurationHealthCheck {
project.runProjectConfigurationHealthCheckWhenEvaluated {
if (!isBrowserConfigured && !isNodejsConfigured) {
project.logger.warn(
"""
@@ -12,7 +12,7 @@ import org.jetbrains.kotlin.gradle.plugin.mpp.KotlinOnlyTarget
import org.jetbrains.kotlin.gradle.plugin.mpp.KotlinOnlyTargetPreset
import org.jetbrains.kotlin.gradle.plugin.statistics.KotlinBuildStatsService
import org.jetbrains.kotlin.gradle.utils.lowerCamelCaseName
import org.jetbrains.kotlin.gradle.utils.runPostEvaluationProjectConfigurationHealthCheck
import org.jetbrains.kotlin.gradle.utils.runProjectConfigurationHealthCheckWhenEvaluated
import org.jetbrains.kotlin.statistics.metrics.StringMetrics
open class KotlinJsIrTargetPreset(
@@ -32,7 +32,7 @@ open class KotlinJsIrTargetPreset(
return project.objects.newInstance(KotlinJsIrTarget::class.java, project, platformType, mixedMode).apply {
this.isMpp = this@KotlinJsIrTargetPreset.isMpp
if (!mixedMode) {
project.runPostEvaluationProjectConfigurationHealthCheck {
project.runProjectConfigurationHealthCheckWhenEvaluated {
if (!isBrowserConfigured && !isNodejsConfigured) {
project.logger.warn(
"""
@@ -74,8 +74,8 @@ internal inline fun Project.runProjectConfigurationHealthCheck(check: Project.()
* ```
* @see runProjectConfigurationHealthCheck
*/
internal inline fun Project.runPostEvaluationProjectConfigurationHealthCheck(crossinline action: Project.() -> Unit) {
internal inline fun Project.runProjectConfigurationHealthCheckWhenEvaluated(crossinline check: Project.() -> Unit) {
whenEvaluated {
runProjectConfigurationHealthCheck(action)
runProjectConfigurationHealthCheck(check)
}
}