Skip to content

Commit 1d5274f

Browse files
committed
Use smaller method definition
1 parent b06061f commit 1d5274f

File tree

1 file changed

+23
-28
lines changed

1 file changed

+23
-28
lines changed

buildSrc/src/main/java/com/google/firebase/gradle/plugins/FireEscapeArtifactPlugin.kt

Lines changed: 23 additions & 28 deletions
Original file line numberDiff line numberDiff line change
@@ -31,53 +31,48 @@ class FireEscapeArtifactPlugin : Plugin<Project> {
3131

3232
override fun apply(target: Project) {
3333
target.afterEvaluate {
34-
if (!supportsMavenPublishing(target)) {
35-
return@afterEvaluate
36-
}
37-
val apiTxtFile = registerApiTxtFileTask(project)
38-
val proguardMappingFile = registerProguardMappingFileTask(project)
39-
val javadoc = registerJavadocTask(project)
34+
if (supportsMavenPublishing(target)) {
35+
val apiTxtFile = registerApiTxtFileTask(project)
36+
val proguardMappingFile = registerProguardMappingFileTask(project)
37+
val javadoc = registerJavadocTask(project)
4038

41-
val zippedArtifact =
42-
project.tasks.register<Zip>("maven") {
43-
from(apiTxtFile)
44-
if (project.isAndroid()) {
45-
from(proguardMappingFile)
39+
val zippedArtifact =
40+
project.tasks.register<Zip>("maven") {
41+
from(apiTxtFile)
42+
if (project.isAndroid()) {
43+
from(proguardMappingFile)
44+
}
4645
}
47-
}
4846

49-
extensions.configure<PublishingExtension> {
50-
publications.getByName<MavenPublication>("mavenAar") {
51-
artifact(zippedArtifact)
52-
artifact(javadoc)
47+
extensions.configure<PublishingExtension> {
48+
publications.getByName<MavenPublication>("mavenAar") {
49+
artifact(zippedArtifact)
50+
artifact(javadoc)
51+
}
5352
}
5453
}
5554
}
5655
}
57-
private fun supportsMavenPublishing(project: Project): Boolean {
58-
return project.plugins.hasPlugin(MavenPublishPlugin::class.java)
59-
}
56+
private fun supportsMavenPublishing(project: Project): Boolean =
57+
project.plugins.hasPlugin(MavenPublishPlugin::class.java)
6058

61-
private fun registerProguardMappingFileTask(project: Project): TaskProvider<Task> {
62-
return project.tasks.register("fireEscapeProguardMapping") {
59+
private fun registerProguardMappingFileTask(project: Project): TaskProvider<Task> =
60+
project.tasks.register("fireEscapeProguardMapping") {
6361
outputs.file(project.fileFromBuildDir("outputs/mapping/release/mapping.txt"))
6462
}
65-
}
6663

67-
private fun registerApiTxtFileTask(project: Project): TaskProvider<Task> {
68-
return project.tasks.register("fireEscapeApiText") {
64+
private fun registerApiTxtFileTask(project: Project): TaskProvider<Task> =
65+
project.tasks.register("fireEscapeApiText") {
6966
dependsOn(JAVADOC_TASK_NAME)
7067
outputs.file(project.fileFromBuildDir("tmp/javadoc/api.txt"))
7168
}
72-
}
7369

74-
private fun registerJavadocTask(project: Project): TaskProvider<Jar> {
75-
return project.tasks.register<Jar>("fireescapeJavadocJar") {
70+
private fun registerJavadocTask(project: Project): TaskProvider<Jar> =
71+
project.tasks.register<Jar>("fireescapeJavadocJar") {
7672
dependsOn(JAVADOC_TASK_NAME)
7773
project.fileFromBuildDir("/docs/javadoc/reference")
7874
include("**/*")
7975
archiveFileName.set("fireescape-javadoc.jar")
8076
archiveClassifier.set("javadoc")
8177
}
82-
}
8378
}

0 commit comments

Comments
 (0)