npm
diff --git a/‎lib/content/package-json.hbs
Copy file name to clipboard
+12Lines changed: 12 additions & 0 deletions b/‎lib/content/package-json.hbs
Copy file name to clipboard
+12Lines changed: 12 additions & 0 deletions
diff --git a/‎lib/release/node-workspace-format.js
Copy file name to clipboard
+3-1Lines changed: 3 additions & 1 deletion b/‎lib/release/node-workspace-format.js
Copy file name to clipboard
+3-1Lines changed: 3 additions & 1 deletion
diff --git a/‎lib/release/release-manager.js
Copy file name to clipboard
+2-1Lines changed: 2 additions & 1 deletion b/‎lib/release/release-manager.js
Copy file name to clipboard
+2-1Lines changed: 2 additions & 1 deletion
diff --git a/‎lib/release/release-please.js
Copy file name to clipboard
+3-1Lines changed: 3 additions & 1 deletion b/‎lib/release/release-please.js
Copy file name to clipboard
+3-1Lines changed: 3 additions & 1 deletion
diff --git a/‎lib/release/semver-versioning-strategy.js
Copy file name to clipboard
+2-1Lines changed: 2 additions & 1 deletion b/‎lib/release/semver-versioning-strategy.js
Copy file name to clipboard
+2-1Lines changed: 2 additions & 1 deletion
diff --git a/‎lib/util/import-or-require.js
Copy file name to clipboard
+2-2Lines changed: 2 additions & 2 deletions b/‎lib/util/import-or-require.js
Copy file name to clipboard
+2-2Lines changed: 2 additions & 2 deletions
diff --git a/‎package.json
Copy file name to clipboard
+12-10Lines changed: 12 additions & 10 deletions b/‎package.json
Copy file name to clipboard
+12-10Lines changed: 12 additions & 10 deletions
diff --git a/‎tap-snapshots/test/apply/files-snapshots.js.test.cjs
Copy file name to clipboard
+9-9Lines changed: 9 additions & 9 deletions b/‎tap-snapshots/test/apply/files-snapshots.js.test.cjs
Copy file name to clipboard
+9-9Lines changed: 9 additions & 9 deletions
diff --git a/‎tap-snapshots/test/apply/source-snapshots.js.test.cjs
Copy file name to clipboard
+5-5Lines changed: 5 additions & 5 deletions b/‎tap-snapshots/test/apply/source-snapshots.js.test.cjs
Copy file name to clipboard
+5-5Lines changed: 5 additions & 5 deletions
diff --git a/‎tap-snapshots/test/bin/check.js.test.cjs
Copy file name to clipboard
+1-1Lines changed: 1 addition & 1 deletion b/‎tap-snapshots/test/bin/check.js.test.cjs
Copy file name to clipboard
+1-1Lines changed: 1 addition & 1 deletion
@@ -40,13 +40,24 @@
   },
   "templateVersion": {{{ del }}},
   "standard": {{{ del }}},
+  "nyc": {{#if tap18}}{
+    "exclude": {{{ json workspaceGlobs }}}
+  }{{else}}{{{ del }}}{{/if}},
   "tap": {
+    {{#if tap18}}
+    {{#if workspaceGlobs}}
+    "exclude": {{{ json workspaceGlobs }}},
+    {{/if}}
+    "test-ignore": {{{ del }}},
+    {{else}}
+    "exclude": {{{ del }}},
     {{#if workspacePaths}}
       "exclude": {{#if tap18}}[
         "{{ join workspaceGlobs "," }}"
         ]{{else }}{{{ del }}}{{/if}},
       "test-ignore": {{#if tap18}}{{{ del }}}{{else}}"^({{ join workspacePaths "|" }})/"{{/if}},
     {{/if}}
+    {{/if}}
     {{#if typescript}}
     {{#if tap16}}
     "coverage": false,
@@ -57,6 +68,7 @@
     ],
     {{/if}}
     {{/if}}
+    "show-full-coverage": {{#if tap18}}true{{else}}{{{ del }}}{{/if}},
     "nyc-arg": {{#if tap18}}{{{ del }}}{{else}}[
       {{#each workspaceGlobs}}
       "--exclude",
 
@@ -5,7 +5,8 @@ const { TagName } = require('release-please/build/src/util/tag-name.js')
 const { ROOT_PROJECT_PATH } = require('release-please/build/src/manifest.js')
 const { DEPS, link, wrapSpecs, list } = require('./util.js')
 
-/* istanbul ignore next: TODO fix flaky tests and enable coverage */
+/* TODO fix flaky tests and enable coverage */
+/* c8 ignore start */
 module.exports = class extends ManifestPlugin {
   static WORKSPACE_MESSAGE = (name, version) => `${DEPS}(workspace): ${name}@${version}`
 
@@ -95,3 +96,4 @@ module.exports = class extends ManifestPlugin {
     }
   }
 }
+/* c8 ignore end */
@@ -165,13 +165,14 @@ class ReleaseManager {
       // If the url fails with anything but a 404 we want the process to blow
       // up because that means something is very wrong. This is a rare edge
       // case that isn't worth testing.
-      /* istanbul ignore else */
+
       if (r.statusCode === 200) {
         this.#info('Found release process from wiki')
         return r.body.text()
       } else if (r.statusCode === 404) {
         this.#info('No release process found in wiki, falling back to default process')
         return this.#getReleaseSteps()
+        /* c8 ignore next 3 */
       } else {
         throw new Error(`Release process fetch failed with status: ${r.statusCode}`)
       }
 
@@ -9,7 +9,8 @@ const NodeWorkspaceFormat = require('./node-workspace-format.js')
 const { getPublishTag, noop } = require('./util.js')
 const { SemverVersioningStrategy } = require('./semver-versioning-strategy.js')
 
-/* istanbul ignore next: TODO fix flaky tests and enable coverage */
+/* TODO fix flaky tests and enable coverage */
+/* c8 ignore start */
 class ReleasePlease {
   #token
   #owner
@@ -163,5 +164,6 @@ class ReleasePlease {
     return releases.length ? releases : null
   }
 }
+/* c8 ignore end */
 
 module.exports = ReleasePlease
@@ -59,10 +59,11 @@ class SemverVersioningStrategy {
     const releaseType = parseCommits(commits)
     const addPreIfNeeded = prerelease ? `pre${releaseType}` : releaseType
     const version = inc(currentVersion.toString(), addPreIfNeeded, tag)
-    /* istanbul ignore next */
+    /* c8 ignore start */
     if (!version) {
       throw new Error('Could not bump version')
     }
+    /* c8 ignore stop */
     return Version.parse(version)
   }
 }
 
@@ -15,12 +15,12 @@ const importOrRequire = async path => {
   try {
     content = require(path)
     // this is for node 22+
-    // istanbul ignore next
+    /* c8 ignore start */
     if (content.__esModule) {
       return content.default
     }
+    /* c8 ignore end */
   } catch {
-    // istanbul ignore next
     try {
       // this is for node under 20
       const results = await import(pathToFileURL(path))
 
@@ -73,20 +73,17 @@
     "eslint-config-prettier": "^9.1.0",
     "nock": "^13.3.8",
     "prettier": "^3.2.5",
-    "tap": "^16.0.0"
+    "tap": "^21.0.1"
   },
   "tap": {
     "timeout": 600,
-    "nyc-arg": [
-      "--exclude",
-      "workspace/test-workspace/**",
-      "--exclude",
-      "tap-snapshots/**"
-    ],
-    "test-ignore": "^(workspace/test-workspace)/",
     "node-arg": [
       "--no-experimental-fetch"
-    ]
+    ],
+    "exclude": [
+      "workspace/test-workspace/**"
+    ],
+    "show-full-coverage": true
   },
   "templateOSS": {
     "//@npmcli/template-oss": "This file is partially managed by @npmcli/template-oss. Edits may be overwritten.",
@@ -98,5 +95,10 @@
   },
   "workspaces": [
     "workspace/test-workspace"
-  ]
+  ],
+  "nyc": {
+    "exclude": [
+      "workspace/test-workspace/**"
+    ]
+  }
 }
@@ -5,7 +5,7 @@
  * Make sure to inspect the output below.  Do not ignore changes!
  */
 'use strict'
-exports[`test/apply/files-snapshots.js TAP private workspace > expect resolving Promise 1`] = `
+exports[`test/apply/files-snapshots.js > TAP > private workspace > promise must resolve to match snapshot 1`] = `
 .commitlintrc.js
 .eslintrc.js
 .github/actions/create-check/action.yml
@@ -42,7 +42,7 @@ workspaces/b/.gitignore
 workspaces/b/package.json
 `
 
-exports[`test/apply/files-snapshots.js TAP turn off add/rm types > expect resolving Promise 1`] = `
+exports[`test/apply/files-snapshots.js > TAP > turn off add/rm types > promise must resolve to match snapshot 1`] = `
 .commitlintrc.js
 .github/actions/create-check/action.yml
 .github/actions/install-latest-npm/action.yml
@@ -65,7 +65,7 @@ package.json
 release-please-config.json
 `
 
-exports[`test/apply/files-snapshots.js TAP turn off module > expect resolving Promise 1`] = `
+exports[`test/apply/files-snapshots.js > TAP > turn off module > promise must resolve to match snapshot 1`] = `
 .commitlintrc.js
 .github/actions/create-check/action.yml
 .github/actions/install-latest-npm/action.yml
@@ -88,7 +88,7 @@ package.json
 release-please-config.json
 `
 
-exports[`test/apply/files-snapshots.js TAP turn off repo > expect resolving Promise 1`] = `
+exports[`test/apply/files-snapshots.js > TAP > turn off repo > promise must resolve to match snapshot 1`] = `
 .eslintrc.js
 .gitignore
 .npmrc
@@ -98,11 +98,11 @@ package.json
 SECURITY.md
 `
 
-exports[`test/apply/files-snapshots.js TAP turn off root > expect resolving Promise 1`] = `
+exports[`test/apply/files-snapshots.js > TAP > turn off root > promise must resolve to match snapshot 1`] = `
 package.json
 `
 
-exports[`test/apply/files-snapshots.js TAP turn off specific files > expect resolving Promise 1`] = `
+exports[`test/apply/files-snapshots.js > TAP > turn off specific files > promise must resolve to match snapshot 1`] = `
 .eslintrc.yml
 .github/actions/create-check/action.yml
 .github/actions/install-latest-npm/action.yml
@@ -131,7 +131,7 @@ release-please-config.json
 SECURITY.md
 `
 
-exports[`test/apply/files-snapshots.js TAP workspaces > expect resolving Promise 1`] = `
+exports[`test/apply/files-snapshots.js > TAP > workspaces > promise must resolve to match snapshot 1`] = `
 .github/actions/create-check/action.yml
 .github/actions/install-latest-npm/action.yml
 .github/dependabot.yml
@@ -158,7 +158,7 @@ workspaces/d/.gitignore
 workspaces/d/package.json
 `
 
-exports[`test/apply/files-snapshots.js TAP workspaces only (like npm/cli) > expect resolving Promise 1`] = `
+exports[`test/apply/files-snapshots.js > TAP > workspaces only (like npm/cli) > promise must resolve to match snapshot 1`] = `
 .github/actions/create-check/action.yml
 .github/actions/install-latest-npm/action.yml
 .github/dependabot.yml
@@ -182,7 +182,7 @@ workspaces/b/.gitignore
 workspaces/b/package.json
 `
 
-exports[`test/apply/files-snapshots.js TAP workspaces with relative content path > expect resolving Promise 1`] = `
+exports[`test/apply/files-snapshots.js > TAP > workspaces with relative content path > promise must resolve to match snapshot 1`] = `
 content_dir/index.js
 content_dir2/index.js
 package.json
 
@@ -5,7 +5,7 @@
  * Make sure to inspect the output below.  Do not ignore changes!
  */
 'use strict'
-exports[`test/apply/source-snapshots.js TAP root only > expect resolving Promise 1`] = `
+exports[`test/apply/source-snapshots.js > TAP > root only > promise must resolve to match snapshot 1`] = `
 .commitlintrc.js
 ========================================
 /* This file is automatically added by @npmcli/template-oss. Do not edit. */
@@ -1426,7 +1426,7 @@ If the vulnerability you have found is [in scope for the GitHub Bug Bounty Progr
 Thanks for helping make GitHub safe for everyone.
 `
 
-exports[`test/apply/source-snapshots.js TAP with content path > expect resolving Promise 1`] = `
+exports[`test/apply/source-snapshots.js > TAP > with content path > promise must resolve to match snapshot 1`] = `
 content_dir/index.js
 ========================================
 module.exports={}
@@ -1444,7 +1444,7 @@ package.json
 }
 `
 
-exports[`test/apply/source-snapshots.js TAP with workspaces > expect resolving Promise 1`] = `
+exports[`test/apply/source-snapshots.js > TAP > with workspaces > promise must resolve to match snapshot 1`] = `
 .commitlintrc.js
 ========================================
 /* This file is automatically added by @npmcli/template-oss. Do not edit. */
@@ -3272,7 +3272,7 @@ workspaces/b/package.json
 }
 `
 
-exports[`test/apply/source-snapshots.js TAP workspaces only > expect resolving Promise 1`] = `
+exports[`test/apply/source-snapshots.js > TAP > workspaces only > promise must resolve to match snapshot 1`] = `
 .github/actions/create-check/action.yml
 ========================================
 # This file is automatically added by @npmcli/template-oss. Do not edit.
@@ -4636,7 +4636,7 @@ workspaces/b/package.json
 }
 `
 
-exports[`test/apply/source-snapshots.js TAP workspaces with nested content path > expect resolving Promise 1`] = `
+exports[`test/apply/source-snapshots.js > TAP > workspaces with nested content path > promise must resolve to match snapshot 1`] = `
 content_dir/index.js
 ========================================
 module.exports={}
 
@@ -5,7 +5,7 @@
  * Make sure to inspect the output below.  Do not ignore changes!
  */
 'use strict'
-exports[`test/bin/check.js TAP problems > must match snapshot 1`] = `
+exports[`test/bin/check.js > TAP > problems > must match snapshot 1`] = `
 
 Some problems were detected:
Original file line number	Diff line number	Diff line change
`@@ -5,7 +5,8 @@ const { TagName } = require('release-please/build/src/util/tag-name.js')`
`5`	`5`	`const { ROOT_PROJECT_PATH } = require('release-please/build/src/manifest.js')`
`6`	`6`	`const { DEPS, link, wrapSpecs, list } = require('./util.js')`
`7`	`7`
`8`		`-/* istanbul ignore next: TODO fix flaky tests and enable coverage */`
	`8`	`+/* TODO fix flaky tests and enable coverage */`
	`9`	`+/* c8 ignore start */`
`9`	`10`	`module.exports = class extends ManifestPlugin {`
`10`	`11`	static WORKSPACE_MESSAGE = (name, version) => `${DEPS}(workspace): ${name}@${version}`
`11`	`12`
`@@ -95,3 +96,4 @@ module.exports = class extends ManifestPlugin {`
`95`	`96`	`}`
`96`	`97`	`}`
`97`	`98`	`}`
	`99`	`+/* c8 ignore end */`
Original file line number	Diff line number	Diff line change
`@@ -9,7 +9,8 @@ const NodeWorkspaceFormat = require('./node-workspace-format.js')`
`9`	`9`	`const { getPublishTag, noop } = require('./util.js')`
`10`	`10`	`const { SemverVersioningStrategy } = require('./semver-versioning-strategy.js')`
`11`	`11`
`12`		`-/* istanbul ignore next: TODO fix flaky tests and enable coverage */`
	`12`	`+/* TODO fix flaky tests and enable coverage */`
	`13`	`+/* c8 ignore start */`
`13`	`14`	`class ReleasePlease {`
`14`	`15`	`#token`
`15`	`16`	`#owner`
`@@ -163,5 +164,6 @@ class ReleasePlease {`
`163`	`164`	`return releases.length ? releases : null`
`164`	`165`	`}`
`165`	`166`	`}`
	`167`	`+/* c8 ignore end */`
`166`	`168`
`167`	`169`	`module.exports = ReleasePlease`
Original file line number	Diff line number	Diff line change
`@@ -59,10 +59,11 @@ class SemverVersioningStrategy {`
`59`	`59`	`const releaseType = parseCommits(commits)`
`60`	`60`	const addPreIfNeeded = prerelease ? `pre${releaseType}` : releaseType
`61`	`61`	`const version = inc(currentVersion.toString(), addPreIfNeeded, tag)`
`62`		`- /* istanbul ignore next */`
	`62`	`+ /* c8 ignore start */`
`63`	`63`	`if (!version) {`
`64`	`64`	`throw new Error('Could not bump version')`
`65`	`65`	`}`
	`66`	`+ /* c8 ignore stop */`
`66`	`67`	`return Version.parse(version)`
`67`	`68`	`}`
`68`	`69`	`}`
Original file line number	Diff line number	Diff line change
`@@ -5,7 +5,7 @@`
`5`	`5`	`* Make sure to inspect the output below. Do not ignore changes!`
`6`	`6`	`*/`
`7`	`7`	`'use strict'`
`8`		-exports[`test/apply/source-snapshots.js TAP root only > expect resolving Promise 1`] = `
	`8`	+exports[`test/apply/source-snapshots.js > TAP > root only > promise must resolve to match snapshot 1`] = `
`9`	`9`	`.commitlintrc.js`
`10`	`10`	`========================================`
`11`	`11`	`/* This file is automatically added by @npmcli/template-oss. Do not edit. */`
`@@ -1426,7 +1426,7 @@ If the vulnerability you have found is [in scope for the GitHub Bug Bounty Progr`
`1426`	`1426`	`Thanks for helping make GitHub safe for everyone.`
`1427`	`1427`	`
`1428`	`1428`
`1429`		-exports[`test/apply/source-snapshots.js TAP with content path > expect resolving Promise 1`] = `
	`1429`	+exports[`test/apply/source-snapshots.js > TAP > with content path > promise must resolve to match snapshot 1`] = `
`1430`	`1430`	`content_dir/index.js`
`1431`	`1431`	`========================================`
`1432`	`1432`	`module.exports={}`
`@@ -1444,7 +1444,7 @@ package.json`
`1444`	`1444`	`}`
`1445`	`1445`	`
`1446`	`1446`
`1447`		-exports[`test/apply/source-snapshots.js TAP with workspaces > expect resolving Promise 1`] = `
	`1447`	+exports[`test/apply/source-snapshots.js > TAP > with workspaces > promise must resolve to match snapshot 1`] = `
`1448`	`1448`	`.commitlintrc.js`
`1449`	`1449`	`========================================`
`1450`	`1450`	`/* This file is automatically added by @npmcli/template-oss. Do not edit. */`
`@@ -3272,7 +3272,7 @@ workspaces/b/package.json`
`3272`	`3272`	`}`
`3273`	`3273`	`
`3274`	`3274`
`3275`		-exports[`test/apply/source-snapshots.js TAP workspaces only > expect resolving Promise 1`] = `
	`3275`	+exports[`test/apply/source-snapshots.js > TAP > workspaces only > promise must resolve to match snapshot 1`] = `
`3276`	`3276`	`.github/actions/create-check/action.yml`
`3277`	`3277`	`========================================`
`3278`	`3278`	`# This file is automatically added by @npmcli/template-oss. Do not edit.`
`@@ -4636,7 +4636,7 @@ workspaces/b/package.json`
`4636`	`4636`	`}`
`4637`	`4637`	`
`4638`	`4638`
`4639`		-exports[`test/apply/source-snapshots.js TAP workspaces with nested content path > expect resolving Promise 1`] = `
	`4639`	+exports[`test/apply/source-snapshots.js > TAP > workspaces with nested content path > promise must resolve to match snapshot 1`] = `
`4640`	`4640`	`content_dir/index.js`
`4641`	`4641`	`========================================`
`4642`	`4642`	`module.exports={}`