feat: lint pr head, base and add optional url

src/accountConfigs/christophehurpeau.ts

@@ -14,7 +14,7 @@ const config: Config<'dev', never> = {
       {
         regExp:
           // eslint-disable-next-line unicorn/no-unsafe-regex
-          /^(revert: )?(build|chore|ci|docs|feat|fix|perf|refactor|style|test)(\(([/a-z-]*)\))?(!)?:\s/,
+          /^(?<revert>revert: )?(?<type>build|chore|ci|docs|feat|fix|perf|refactor|style|test)(?<scope>\([/a-z-]+\)?((?=:\s)|(?=!:\s)))?(?<breaking>!)?(?<subject>:\s.*)$/,
         error: {
           title: 'Title does not match commitlint conventional',
           summary:

src/accountConfigs/ornikar.ts

@@ -14,9 +14,8 @@ const config: Config<'dev' | 'design', 'ops' | 'frontends' | 'backends'> = {
   parsePR: {
     title: [
       {
-        regExp:
-          // eslint-disable-next-line unicorn/no-unsafe-regex
-          /^(revert: )?(build|chore|ci|docs|feat|fix|perf|refactor|style|test)(\(([/A-Za-z-]*)\))?:\s/,
+        // eslint-disable-next-line unicorn/no-unsafe-regex
+        regExp: /^(?<revert>revert: )?(?<type>build|chore|ci|docs|feat|fix|perf|refactor|style|test)(?<scope>\([/A-Za-z-]+\)?((?=:\s)|(?=!:\s)))?(?<breaking>!)?(?<subject>:\s.*)$/,
         error: {
           title: 'Title does not match commitlint conventional',
           summary:
@@ -27,7 +26,7 @@ const config: Config<'dev' | 'design', 'ops' | 'frontends' | 'backends'> = {
         bot: false,
         regExp: /\s([A-Z][\dA-Z]+-(\d+)|\[no issue])$/,
         error: {
-          title: 'Title does not have JIRA issue',
+          title: 'Title does not have Jira issue',
           summary: 'The PR title should end with ONK-0000, or [no issue]',
         },
         status: 'jira-issue',
@@ -48,6 +47,28 @@ const config: Config<'dev' | 'design', 'ops' | 'frontends' | 'backends'> = {
         },
       },
     ],
+    head: [
+      {
+        bot: false,
+        // eslint-disable-next-line unicorn/no-unsafe-regex
+        regExp: /^(?<revert>revert-\d+-)?(?<type>build|chore|ci|docs|feat|fix|perf|refactor|style|test)(?<scope>\/[a-z-]+)?\/(?<breaking>!)?(?<subject>.*)-(?<jiraIssue>[A-Z][\dA-Z]+-(\d+))$/,
+        warning: true,
+        error: {
+          title: 'Branch name does not match commitlint conventional',
+          summary: '',
+        },
+      },
+    ],
+    base: [
+      {
+        regExp: /^(master|main)$/,
+        error: {
+          title: 'PR to branches other than main is not recommended',
+          summary:
+            'https://ornikar.atlassian.net/wiki/spaces/TECH/pages/2221900272/Should+I+make+a+feature-branch+or+not',
+        },
+      },
+    ],
   },
 
   botUsers: ['michael-robot'],

src/accountConfigs/types.ts

@@ -10,6 +10,7 @@ export interface StatusInfo {
 export interface StatusError {
   title: string;
   summary: string;
+  url?: string;
 }
 
 export interface Group {
@@ -26,13 +27,16 @@ export interface ParsePRRule {
   bot?: false;
   regExp: RegExp;
   error: StatusError;
+  warning?: boolean;
 
   status?: string;
   statusInfoFromMatch?: (match: RegExpMatchArray) => StatusInfo;
 }
 
 export interface ParsePR {
   title: ParsePRRule[];
+  head?: ParsePRRule[];
+  base?: ParsePRRule[];
 }
 
 export interface LabelDescriptor {

src/context/utils.ts

@@ -1,8 +1,6 @@
 import createEmojiRegex from 'emoji-regex';
 
-export const getKeys = <T extends Record<keyof T, unknown>>(
-  o: T,
-): (keyof T)[] => Object.keys(o) as (keyof T)[];
+export const getKeys = <T>(o: T): (keyof T)[] => Object.keys(o) as (keyof T)[];
 
 const emojiRegex = createEmojiRegex();