[tasks/lib/notice] join all notices with the same separator

elastic · May 23, 2017 · a80c43e · a80c43e
1 parent e5a4839
commit a80c43e
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 8 deletions.
diff --git a/tasks/lib/notice/base_notice.txt b/tasks/lib/notice/base_notice.txt
@@ -56,4 +56,3 @@ THE SOFTWARE.
 ---
 This product bundles geohash.js which is available under a
 "MIT" license.  For details, see src/ui/public/utils/decode_geo_hash.js.
----
diff --git a/tasks/lib/notice/notice.js b/tasks/lib/notice/notice.js
@@ -1,7 +1,7 @@
 import { resolve } from 'path';
 import { readFileSync } from 'fs';
 
-import { generatePackagesNoticeText } from './packages_notice';
+import { generatePackageNoticeTexts } from './packages_notice';
 import { generateNodeNoticeText } from './node_notice';
 
 const BASE_NOTICE = resolve(__dirname, './base_notice.txt');
@@ -22,7 +22,7 @@ export async function generateNoticeText(options = {}) {
   const { packages, nodeDir } = options;
   return [
     readFileSync(BASE_NOTICE, 'utf8'),
-    await generatePackagesNoticeText(packages),
+    ...(await generatePackageNoticeTexts(packages)),
     generateNodeNoticeText(nodeDir)
-  ].join('');
+  ].join('\n---\n');
 }
diff --git a/tasks/lib/notice/packages_notice.js b/tasks/lib/notice/packages_notice.js
@@ -6,8 +6,8 @@ const concatNotices = notices => (
   notices.map(notice => notice.text).join('\n')
 );
 
-export async function generatePackagesNoticeText(packages) {
-  const noticeChunks = await asyncMap(packages, async pkg => {
+export async function generatePackageNoticeTexts(packages) {
+  return await asyncMap(packages, async pkg => {
     const bundledNotices = concatNotices(await getBundledNotices(pkg.directory));
 
     const intro = `This product bundles ${pkg.name}@${pkg.version}`;
@@ -23,6 +23,4 @@ export async function generatePackagesNoticeText(packages) {
 
     return `${intro}${license}${moreInfo}`;
   });
-
-  return noticeChunks.join('\n---\n');
 }