fix(ng-dev): properly account for not included in list (#2322)

Properly handle a "not in" clause in pullapprove.

PR Close #2322

Author: josephperrott

ng-dev/pullapprove/condition_evaluator.ts

@@ -81,7 +81,7 @@ export function convertConditionToFunction(
  */
 function transformExpressionToJs(expression: string): string {
   return expression
-    .replace(/^(.+)\s+not in\s+(\[.+\])$/, '!$2.some(x => $1.matchesAny || $1 == x)')
+    .replace(/^(.+)\s+not in\s+(\[.+\])$/, '!$2.includes($1)')
     .replace(/^(.+)\s+in\s+(.+)$/, '$2.some(x => $1.matchesAny || $1 == x)')
     .replace(/^(.+)\s+not in\s+(.+)$/, '!$2.includes($1)')
     .replace(/^(.+)\s+in\s+(.+)$/, '$2.includes($1)')