fix(publish): pass dryRun: true to libnpmpack so it doesnt write a tarball

nlf · nlf · commit e2f1f7b045a3 · 2022-02-08T13:48:23.000-08:00
diff --git a/lib/commands/publish.js b/lib/commands/publish.js
@@ -83,7 +83,8 @@ class Publish extends BaseCommand {
       })
     }
 
-    const tarballData = await pack(spec, opts)
+    // we pass dryRun: true to libnpmpack so it doesn't write the file to disk
+    const tarballData = await pack(spec, { ...opts, dryRun: true })
     const pkgContents = await getContents(manifest, tarballData)
 
     // The purpose of re-reading the manifest is in case it changed,

Original file line number	Diff line number	Diff line change
`@@ -83,7 +83,8 @@ class Publish extends BaseCommand {`
`83`	`83`	`})`
`84`	`84`	`}`
`85`	`85`
`86`		`- const tarballData = await pack(spec, opts)`
	`86`	`+ // we pass dryRun: true to libnpmpack so it doesn't write the file to disk`
	`87`	`+ const tarballData = await pack(spec, { ...opts, dryRun: true })`
`87`	`88`	`const pkgContents = await getContents(manifest, tarballData)`
`88`	`89`
`89`	`90`	`// The purpose of re-reading the manifest is in case it changed,`