Merge pull request #533 from Financial-Times/chores/add-output-quotes

Add quotes for output messages 🐿 v2.10.3
Financial-Times · Nov 21, 2018 · c0b3dea · c0b3dea
2 parents b551843 + 1c29ea9
commit c0b3dea
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/lib/review-apps.js b/lib/review-apps.js
@@ -127,12 +127,12 @@ const waitForReviewAppBuild = ({ commit, minTimeout = MIN_TIMEOUT }) => async (a
 			const build = builds.find(({ source_blob: { version } }) => version === commit);
 
 			if (!build) {
-				throw new Error(`No review app build found for app id ${appId}, commit ${commit}`);
+				throw new Error(`No review app build found for app id '${appId}';, commit '${commit}'`);
 			}
 
 			const { status } = build;
 			if (status !== BUILD_STATUS_SUCCEEDED) {
-				throw new Error(`Review app build for app id ${appId} (commit ${commit}) not done yet: ${status}`);
+				throw new Error(`Review app build for app id '${appId}' (commit '${commit}') not done yet: ${status}`);
 			}
 
 			return build;

diff --git a/tasks/review-app.js b/tasks/review-app.js
@@ -15,7 +15,7 @@ async function task (appName, options) {
 		.then(res => {
 			const { status } = res;
 			if (status === 409) {
-				console.error(`Review app already created for ${branch} branch. Using existing review app for build.`); // eslint-disable-line no-console
+				console.error(`Review app already created for '${branch}' branch. Using existing review app for build.`); // eslint-disable-line no-console
 				return findCreatedReviewApp({
 					pipelineId,
 					branch