feat(focus): allow multiple lib/package focus in OSS projects

nstudio · Sep 8, 2020 · 841ab5e · 841ab5e
1 parent f12b635
commit 841ab5e
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 4 deletions.
diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md
@@ -110,6 +110,7 @@ The scope must be one of the following:
 
 - angular - anything Angular specific
 - core - anything xplat core specific
+- focus - anything related to focus tools
 - docs - anything related to docs infrastructure
 - electron - anything Electron specific
 - ionic - anything Ionic specific

diff --git a/packages/focus/src/utils/index.ts b/packages/focus/src/utils/index.ts
@@ -122,14 +122,14 @@ export namespace FocusHelpers {
             const handledPlatforms: any = {};
             for (const target of platforms) {
               for (const p of options.allLibs) {
-                if (!handledPlatforms[target]) {
-                  const libName = p.replace('**/libs/', '');
+                const libName = p.replace('**/libs/', '');
+                if (!handledPlatforms[libName]) {
                   userUpdates[p] = libName === target ? false : true;
                 }
               }
               for (const p of options.allPackages) {
-                if (!handledPlatforms[target]) {
-                  const packageName = p.replace('**/packages/', '');
+                const packageName = p.replace('**/packages/', '');
+                if (!handledPlatforms[packageName]) {
                   userUpdates[p] = packageName === target ? false : true;
                 }
               }