chore: Update prettier config and reformat files

Author: kevinrenskers

.eslintrc.js

@@ -12,7 +12,7 @@ module.exports = {
     sourceType: "module",
   },
   rules: {
-    "prettier/prettier": ["error", { printWidth: 200 }],
+    "no-var": "error",
     "prefer-const": "error",
     eqeqeq: "error",
     "no-useless-return": "error",

.prettierrc

@@ -0,0 +1,7 @@
+{
+  "printWidth": 140,
+  "htmlWhitespaceSensitivity": "strict",
+  "arrowParens": "avoid",
+  "bracketSameLine": true,
+  "trailingComma": "es5"
+}

src/defaultConfig.js

@@ -18,7 +18,7 @@ const DEFAULT_CONFIG = {
   renderTypeSection: function (label, commits) {
     let text = `\n## ${label}\n`;
 
-    commits.forEach((commit) => {
+    commits.forEach(commit => {
       const scope = commit.scope ? `**${commit.scope}:** ` : "";
       text += `- ${scope}${commit.subject}\n`;
     });
@@ -29,7 +29,7 @@ const DEFAULT_CONFIG = {
   renderNotes: function (notes) {
     let text = `\n## BREAKING CHANGES\n`;
 
-    notes.forEach((note) => {
+    notes.forEach(note => {
       text += `- due to [${note.commit.sha.substr(0, 6)}](${note.commit.url}): ${note.commit.subject}\n\n`;
       text += `${note.text}\n\n`;
     });

src/generateChangelog.js

@@ -6,30 +6,30 @@ function generateChangelog(releaseName, commitObjects, config) {
   let changes = "";
 
   commitsByType
-    .filter((obj) => {
+    .filter(obj => {
       return !config.excludeTypes.includes(obj.type);
     })
-    .forEach((obj) => {
+    .forEach(obj => {
       const niceType = translateType(obj.type, config.types);
       changes += config.renderTypeSection(niceType, obj.commits);
     });
 
   // Find all the notes of all the commits of all the types
   const notes = commitsByType
-    .flatMap((obj) => {
+    .flatMap(obj => {
       return obj.commits
-        .map((commit) => {
+        .map(commit => {
           if (commit.notes && commit.notes.length) {
-            return commit.notes.map((note) => {
+            return commit.notes.map(note => {
               const noteObj = note;
               noteObj.commit = commit;
               return noteObj;
             });
           }
         })
-        .filter((o) => o);
+        .filter(o => o);
     })
-    .flatMap((o) => o);
+    .flatMap(o => o);
 
   if (notes.length) {
     changes += config.renderNotes(notes);

src/groupByType.js

@@ -2,7 +2,7 @@ function groupByType(commits, typeConfig) {
   // First, group all the commits by their types.
   // We end up with a dictionary where the key is the type, and the values is an array of commits.
   const byType = {};
-  commits.forEach((commit) => {
+  commits.forEach(commit => {
     if (!byType[commit.type]) {
       byType[commit.type] = [];
     }
@@ -12,7 +12,7 @@ function groupByType(commits, typeConfig) {
   // Turn that dictionary into an array of objects,
   // where the key is the type, and the values is an array of commits.
   const byTypeArray = [];
-  Object.keys(byType).forEach((key) => {
+  Object.keys(byType).forEach(key => {
     byTypeArray.push({
       type: key,
       commits: byType[key],
@@ -21,9 +21,9 @@ function groupByType(commits, typeConfig) {
 
   // And now we sort that array using the TYPES object.
   byTypeArray.sort((a, b) => {
-    let aOrder = typeConfig.findIndex((t) => t.types.includes(a.type));
+    let aOrder = typeConfig.findIndex(t => t.types.includes(a.type));
     if (aOrder === -1) aOrder = 999;
-    let bOrder = typeConfig.findIndex((t) => t.types.includes(b.type));
+    let bOrder = typeConfig.findIndex(t => t.types.includes(b.type));
     if (bOrder === -1) bOrder = 999;
     return aOrder - bOrder;
   });

src/index.js

@@ -44,7 +44,7 @@ async function run() {
   const tags = await octokit.paginate("GET /repos/{owner}/{repo}/tags", { owner, repo });
 
   const validSortedTags = tags
-    .filter((t) => validate(t.name))
+    .filter(t => validate(t.name))
     .sort((a, b) => {
       return compareVersions(a.name, b.name);
     })
@@ -94,14 +94,14 @@ async function run() {
   // Parse every commit, getting the type, turning PR numbers into links, etc
   const commitObjects = await Promise.all(
     result.data.commits
-      .map(async (commit) => {
+      .map(async commit => {
         const commitObj = await parseCommitMessage(commit.commit.message, `https://github.com/${owner}/${repo}`, fetchUserFunc);
         commitObj.sha = commit.sha;
         commitObj.url = commit.html_url;
         commitObj.author = commit.author;
         return commitObj;
       })
-      .filter((m) => m !== false)
+      .filter(m => m !== false)
   );
 
   // And generate the changelog

src/translateType.js

@@ -1,5 +1,5 @@
 function translateType(type, typeConfig) {
-  const foundType = typeConfig.find((t) => t.types.includes(type));
+  const foundType = typeConfig.find(t => t.types.includes(type));
   if (foundType) {
     return foundType.label;
   }

test/parseCommitMessage.spec.js

@@ -29,7 +29,10 @@ describe("parseCommitMessage", () => {
   it("should parse basic feat with a PR number", async () => {
     const result = await parseCommitMessage("feat: This is a feature [#1]", "https://github.com/loopwerk/tag-changelog", fetchUserFunc);
 
-    assert.strictEqual(result.subject, "This is a feature [[#1](https://github.com/loopwerk/tag-changelog/pull/1) by [kevinrenskers](https://github.com/kevinrenskers)]");
+    assert.strictEqual(
+      result.subject,
+      "This is a feature [[#1](https://github.com/loopwerk/tag-changelog/pull/1) by [kevinrenskers](https://github.com/kevinrenskers)]"
+    );
     assert.strictEqual(result.type, "feat");
   });
 
@@ -65,7 +68,10 @@ describe("parseCommitMessage", () => {
   it("should parse a missing type with a PR number", async () => {
     const result = await parseCommitMessage("This is a commit [#1]", "https://github.com/loopwerk/tag-changelog", fetchUserFunc);
 
-    assert.strictEqual(result.subject, "This is a commit [[#1](https://github.com/loopwerk/tag-changelog/pull/1) by [kevinrenskers](https://github.com/kevinrenskers)]");
+    assert.strictEqual(
+      result.subject,
+      "This is a commit [[#1](https://github.com/loopwerk/tag-changelog/pull/1) by [kevinrenskers](https://github.com/kevinrenskers)]"
+    );
     assert.strictEqual(result.type, "other");
   });