fix(config): include the dry-run param from input options

be sure to include the dry-run flag from semantic release if provided. default to false
esatterwhite · Mar 22, 2024 · 2e18e69 · 2e18e69
1 parent b5b3415
commit 2e18e69
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 0 deletions.
diff --git a/lib/build-config.js b/lib/build-config.js
@@ -28,6 +28,7 @@ async function buildConfig(build_id, config, context) {
   , dockerNetwork: network = 'default'
   , dockerAutoClean: clean = true
   , dockerBuildQuiet: quiet = true
+  , 'dry-run': dry_run = false
   } = config
 
   let name = null
@@ -57,6 +58,7 @@ async function buildConfig(build_id, config, context) {
   , publish
   , tags: array.toArray(tags)
   , verifycmd
+  , dry_run
   , args: {
       SRC_DIRECTORY: path.basename(context.cwd)
     , TARGET_PATH: target

diff --git a/test/unit/build-config.js b/test/unit/build-config.js
@@ -45,11 +45,13 @@ test('build-config', async (t) => {
     , context: '.'
     , quiet: true
     , clean: true
+    , dry_run: false
     })
   })
 
   t.test('nested workspace: target resolution', async (tt) => {
     const config = await buildConfig('id', {
+      'dry-run': true
     }, {
       options: {
         root: t.testdirName
@@ -77,6 +79,7 @@ test('build-config', async (t) => {
     , build: 'id'
     , context: '.'
     , quiet: true
+    , dry_run: true
     , clean: true
     })
   })