#1329: fix build --bulk parameter called via CLI

Accenture · May 16, 2024 · 190cf68 · 190cf68
1 parent 2928af9
commit 190cf68
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 3 deletions.
diff --git a/lib/cli.js b/lib/cli.js
@@ -363,8 +363,7 @@ yargs(hideBin(process.argv))
                     type: 'string',
                     alias: 'bt',
                     group: 'Required parameters for build:',
-                    describe: 'the business unit to deploy to; ignored if --bulk is set',
-                    demandOption: true,
+                    describe: 'the business unit to deploy to; required unless --bulk is set',
                 })
                 .option('marketFrom', {
                     type: 'string',
@@ -392,7 +391,14 @@ yargs(hideBin(process.argv))
             Mcdev.setOptions(argv);
             const typeKeyCombo = Mcdev.metadataToTypeKey(argv.metadata);
             if ('undefined' !== typeof typeKeyCombo) {
-                Mcdev.build(argv.buFrom, argv.buTo, typeKeyCombo, argv.marketFrom, argv.marketTo);
+                Mcdev.build(
+                    argv.buFrom,
+                    argv.buTo,
+                    typeKeyCombo,
+                    argv.marketFrom,
+                    argv.marketTo,
+                    argv.bulk
+                );
             }
         },
     })

diff --git a/lib/index.js b/lib/index.js
@@ -797,6 +797,13 @@ class Mcdev {
         marketDefinition,
         bulk
     ) {
+        if (!bulk && !businessUnitDefinition) {
+            Util.logger.error(
+                'Please provide a business unit to deploy to via --buTo or activate --bulk'
+            );
+            return;
+        }
+
         Util.logger.info('mcdev:: Build Template & Build Definition');
         await this.buildTemplate(businessUnitTemplate, typeKeyCombo, null, marketTemplate);
         return bulk