Fix wording

Thanks @hypest
Automattic · Jun 25, 2021 · 26fa081 · 26fa081
1 parent 78af5a8
commit 26fa081
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/org/pr/installable-build.ts b/org/pr/installable-build.ts
@@ -127,8 +127,8 @@ async function getDownloadCommentText(status) {
   if (apkArtifacts.length == 1) {
     return `You can test the changes on this Pull Request by downloading the APK [here](${apkArtifacts[0].url}).`
   } else if (apkArtifacts.length > 1) {
-    const links = apkArtifacts.map(artifact => `- [${artifact.path.split("/").pop()}](${artifact.url})`).join(`\n`)
-    return `You can test the changes on this Pull Request by downloading the APKs:\n${links}.`
+    const links = apkArtifacts.map(artifact => ` - [${artifact.path.split("/").pop()}](${artifact.url})`).join(`\n`)
+    return `You can test the changes on this Pull Request by downloading the APKs:\n${links}`
   }
   return undefined
 }

diff --git a/tests/installable-build-test.ts b/tests/installable-build-test.ts
@@ -174,6 +174,6 @@ describe("installable build handling", () => {
       }
       await installableBuild(webhook)
 
-      expectComment(webhook, `You can test the changes on this Pull Request by downloading the APKs:\n- [file1.apk](${mockedArtifacts[0].url})\n- [file2.apk](${mockedArtifacts[1].url}).`)
+      expectComment(webhook, `You can test the changes on this Pull Request by downloading the APKs:\n - [file1.apk](${mockedArtifacts[0].url})\n - [file2.apk](${mockedArtifacts[1].url})`)
     })
 })