Revert "feat: pr from bot jira-issue"

This reverts commit f2441fd.
christophehurpeau · May 20, 2019 · f0b3b35 · f0b3b35
1 parent 1039e9e
commit f0b3b35
Show file tree

Hide file tree

Showing 7 changed files with 12 additions and 44 deletions.
diff --git a/dist/index-node10-dev.cjs.js b/dist/index-node10-dev.cjs.js
diff --git a/dist/index-node10-dev.cjs.js.map b/dist/index-node10-dev.cjs.js.map
diff --git a/dist/index-node10.cjs.js b/dist/index-node10.cjs.js
diff --git a/dist/index-node10.cjs.js.map b/dist/index-node10.cjs.js.map
diff --git a/src/pr-handlers/actions/editOpenedPR.ts b/src/pr-handlers/actions/editOpenedPR.ts
@@ -53,7 +53,7 @@ export const editOpenedPR = async (
     if (rule.status && rule.statusInfoFromMatch) {
       statuses.push({
         name: rule.status,
-        info: rule.statusInfoFromMatch(match, { bot: isPrFromBot }),
+        info: rule.statusInfoFromMatch(match),
       });
       return false;
     }

diff --git a/src/teamconfigs/ornikar.ts b/src/teamconfigs/ornikar.ts
@@ -22,19 +22,14 @@ const config: Config<'dev' | 'design'> = {
         },
       },
       {
+        bot: false,
         regExp: /\s(ONK-(\d+)|\[no issue])$/,
         error: {
           title: 'Title does not have JIRA issue',
           summary: 'The PR title should end with ONK-0000, or [no issue]',
         },
         status: 'jira-issue',
-        statusInfoFromMatch: (match, { bot }) => {
-          if (bot) {
-            return {
-              title: 'PR from bot',
-              summary: '',
-            };
-          }
+        statusInfoFromMatch: (match) => {
           const issue = match[1];
           if (issue === '[no issue]') {
             return {

diff --git a/src/teamconfigs/types.ts b/src/teamconfigs/types.ts
@@ -15,20 +15,13 @@ export interface Group {
   [userName: string]: string;
 }
 
-export interface StatusInfoFromMatchOptions {
-  bot: boolean;
-}
-
 export interface ParsePRRule {
   bot?: false;
   regExp: RegExp;
   error: StatusError;
 
   status?: string;
-  statusInfoFromMatch?: (
-    match: RegExpMatchArray,
-    options: StatusInfoFromMatchOptions,
-  ) => StatusInfo;
+  statusInfoFromMatch?: (match: RegExpMatchArray) => StatusInfo;
 }
 
 export interface ParsePR {