Build: Add “chore” to commit tags (fixes eslint#5880)

mikesherov · Apr 22, 2016 · 63fb937 · 63fb937
1 parent 495c6c4
commit 63fb937
Show file tree

Hide file tree

Showing 4 changed files with 31 additions and 21 deletions.
diff --git a/Makefile.js b/Makefile.js
@@ -70,7 +70,7 @@ var NODE = "node ", // intentional extra space
     PERF_MULTIFILES_TARGETS = PERF_MULTIFILES_TARGET_DIR + path.sep + "{lib,tests" + path.sep + "lib}" + path.sep + "**" + path.sep + "*.js",
 
     // Regex
-    TAG_REGEX = /^(?:Fix|Update|Breaking|Docs|Build|New|Upgrade):/,
+    TAG_REGEX = /^(?:Breaking|Build|Chore|Docs|Fix|New|Update|Upgrade):/,
     ISSUE_REGEX = /\((?:fixes|refs) #\d+(?:.*(?:fixes|refs) #\d+)*\)$/,
 
     // Settings
@@ -914,6 +914,7 @@ target.checkGitCommit = function() {
                 "    'Build:'",
                 "    'New:'",
                 "    'Upgrade:'",
+                "    'Chore:'",
                 "   Please refer to the contribution guidelines for more details."].join("\n"));
             failed = true;
         }

diff --git a/templates/blogpost.md.ejs b/templates/blogpost.md.ejs
@@ -71,3 +71,10 @@ function outputList(items) {
 <% outputList(changelog.build); %>
 
 <% } %>
+
+<% if (typeof changelog.chore !== "undefined") { %>
+## Chores
+
+<% outputList(changelog.chore); %>
+
+<% } %>
diff --git a/templates/pr-create.md.ejs b/templates/pr-create.md.ejs
@@ -1,6 +1,6 @@
 <%
 
-var FLAG_PATTERN = /^(Fix|Update|New|Breaking|Build|Docs|Upgrade):/,
+var FLAG_PATTERN = /^(Breaking|Build|Chore|Docs|Fix|New|Update|Upgrade):/,
     ISSUE_REF_PATTERN = /\((fixes|refs) #\d+.*?\)$/;
 
 function isValidCommitFlag(log) {

diff --git a/tests/templates/pr-create.md.ejs.js b/tests/templates/pr-create.md.ejs.js
@@ -127,28 +127,30 @@ describe("pr-create.md.ejs", function() {
         assert.ok(result.indexOf("require an issue") === -1);
     });
 
-    it("should not mention missing issue or length check when there's one fix commit", function() {
-        var result = ejs.render(TEMPLATE_TEXT, {
-            payload: {
-                sender: {
-                    login: "nzakas"
+    ["Breaking", "Build", "Chore", "Docs", "Fix", "New", "Update", "Upgrade"].forEach(function(type) {
+        it("should not mention missing issue or length check when there's one " + type + " commit", function() {
+            var result = ejs.render(TEMPLATE_TEXT, {
+                payload: {
+                    sender: {
+                        login: "nzakas"
+                    },
+                    commits: 1
                 },
-                commits: 1
-            },
-            meta: {
-                cla: true,
-                commits: [
-                    {
-                        commit: {
-                            message: "Fix: Foo bar (fixes #1234)\nSome really long string. abcdefghijklmnopqrstuvwxyz abcdefghijklmnopqrstuvwxyz abcdefghijklmnopqrstuvwxyz"
+                meta: {
+                    cla: true,
+                    commits: [
+                        {
+                            commit: {
+                                message: type + ": Foo bar (fixes #1234)\nSome really long string. abcdefghijklmnopqrstuvwxyz abcdefghijklmnopqrstuvwxyz abcdefghijklmnopqrstuvwxyz"
+                            }
                         }
-                    }
-                ]
-            }
-        });
+                    ]
+                }
+            });
 
-        assert.equal(result.trim(), "LGTM");
-        assert.ok(result.indexOf("require an issue") === -1);
+            assert.equal(result.trim(), "LGTM");
+            assert.ok(result.indexOf("require an issue") === -1);
+        });
     });
 
     it("should mention missing issue when there's a missing closing paren", function() {