From 273d9355e35c0b921048ce82b3b636dff38bc0c6 Mon Sep 17 00:00:00 2001
From: Rhys Arkins <rhys@arkins.net>
Date: Sun, 3 May 2020 16:15:22 +0200
Subject: [PATCH] refactor: enabledManagers filtering

---
 lib/workers/repository/extract/index.ts | 17 +++++++++--------
 1 file changed, 9 insertions(+), 8 deletions(-)

diff --git a/lib/workers/repository/extract/index.ts b/lib/workers/repository/extract/index.ts
index 72b8287f0c..65488e8222 100644
--- a/lib/workers/repository/extract/index.ts
+++ b/lib/workers/repository/extract/index.ts
@@ -1,3 +1,4 @@
+import is from '@sindresorhus/is';
 import {
   RenovateConfig,
   getManagerConfig,
@@ -13,14 +14,14 @@ export async function extractAllDependencies(
 ): Promise<Record<string, PackageFile[]>> {
   const extractions: Record<string, PackageFile[]> = {};
   let fileCount = 0;
-  for (const manager of getManagerList()) {
-    if (
-      config.enabledManagers.length &&
-      !config.enabledManagers.includes(manager)
-    ) {
-      logger.debug(`${manager} is not in enabledManagers list - skipping`);
-      continue; // eslint-disable-line
-    }
+  let managerList = getManagerList();
+  if (is.nonEmptyArray(config.enabledManagers)) {
+    logger.debug('Applying enabledManagers filtering');
+    managerList = managerList.filter((manager) =>
+      config.enabledManagers.includes(manager)
+    );
+  }
+  for (const manager of managerList) {
     const managerConfig = getManagerConfig(config, manager);
     let packageFiles = [];
     if (manager === 'regex') {
-- 
GitLab