feat: Pretty-print the generagted HTML table (#190)

Closes #189 Keeps the Table left aligned, and moves the tr and td in by 2 spaces each
all-contributors · Jun 11, 2019 · 68166ec · 68166ec
1 parent 89c3840
commit 68166ec
Show file tree

Hide file tree

Showing 2 changed files with 25 additions and 5 deletions.
diff --git a/src/generate/__tests__/__snapshots__/index.js.snap b/src/generate/__tests__/__snapshots__/index.js.snap
@@ -9,7 +9,13 @@ Description
 These people contributed to the project:
 <!-- ALL-CONTRIBUTORS-LIST:START -->
 <!-- prettier-ignore -->
-<table><tr><td align=\\"center\\">Kent C. Dodds is awesome!</td><td align=\\"center\\">Divjot Singh is awesome!</td><td align=\\"center\\">Jeroen Engels is awesome!</td></tr></table>
+<table>
+  <tr>
+    <td align=\\"center\\">Kent C. Dodds is awesome!</td>
+    <td align=\\"center\\">Divjot Singh is awesome!</td>
+    <td align=\\"center\\">Jeroen Engels is awesome!</td>
+  </tr>
+</table>
 
 <!-- ALL-CONTRIBUTORS-LIST:END -->
 
@@ -25,7 +31,19 @@ Description
 These people contributed to the project:
 <!-- ALL-CONTRIBUTORS-LIST:START -->
 <!-- prettier-ignore -->
-<table><tr><td align=\\"center\\">Kent C. Dodds is awesome!</td><td align=\\"center\\">Kent C. Dodds is awesome!</td><td align=\\"center\\">Kent C. Dodds is awesome!</td><td align=\\"center\\">Kent C. Dodds is awesome!</td><td align=\\"center\\">Kent C. Dodds is awesome!</td></tr><tr><td align=\\"center\\">Kent C. Dodds is awesome!</td><td align=\\"center\\">Kent C. Dodds is awesome!</td></tr></table>
+<table>
+  <tr>
+    <td align=\\"center\\">Kent C. Dodds is awesome!</td>
+    <td align=\\"center\\">Kent C. Dodds is awesome!</td>
+    <td align=\\"center\\">Kent C. Dodds is awesome!</td>
+    <td align=\\"center\\">Kent C. Dodds is awesome!</td>
+    <td align=\\"center\\">Kent C. Dodds is awesome!</td>
+  </tr>
+  <tr>
+    <td align=\\"center\\">Kent C. Dodds is awesome!</td>
+    <td align=\\"center\\">Kent C. Dodds is awesome!</td>
+  </tr>
+</table>
 
 <!-- ALL-CONTRIBUTORS-LIST:END -->
 

diff --git a/src/generate/index.js b/src/generate/index.js
@@ -36,7 +36,9 @@ function injectListBetweenTags(newContent) {
 }
 
 function formatLine(contributors) {
-  return `<td align="center">${contributors.join('</td><td align="center">')}</td>`
+  return `<td align="center">${contributors.join(
+    '</td>\n    <td align="center">',
+  )}</td>`
 }
 
 function generateContributorsList(options, contributors) {
@@ -46,9 +48,9 @@ function generateContributorsList(options, contributors) {
     }),
     _.chunk(options.contributorsPerLine),
     _.map(formatLine),
-    _.join('</tr><tr>'),
+    _.join('\n  </tr>\n  <tr>\n    '),
     newContent => {
-      return `\n<table><tr>${newContent}</tr></table>\n\n`
+      return `\n<table>\n  <tr>\n    ${newContent}\n  </tr>\n</table>\n\n`
     },
   )(contributors)
 }