diff --git a/packages/semver/src/builders/version/builder.e2e.spec.ts b/packages/semver/src/builders/version/builder.e2e.spec.ts index 9ab3f03a9..bf0af0f0c 100644 --- a/packages/semver/src/builders/version/builder.e2e.spec.ts +++ b/packages/semver/src/builders/version/builder.e2e.spec.ts @@ -11,10 +11,6 @@ import { TestingWorkspace, } from './testing'; import { readPackageJson } from './utils/project'; -import { _enableWip } from './version'; - -// @todo get rid of this -_enableWip(); describe('@jscutlery/semver:version', () => { const defaultBuilderOptions: VersionBuilderSchema = { diff --git a/packages/semver/src/builders/version/builder.spec.ts b/packages/semver/src/builders/version/builder.spec.ts index 357ea1d7f..13207371b 100644 --- a/packages/semver/src/builders/version/builder.spec.ts +++ b/packages/semver/src/builders/version/builder.spec.ts @@ -11,7 +11,6 @@ import { createFakeContext } from './testing'; import { tryBump } from './utils/try-bump'; import * as workspace from './utils/workspace'; import { getPackageFiles, getProjectRoots } from './utils/workspace'; -import { _enableWip } from './version'; jest.mock('child_process'); jest.mock('@lerna/child-process'); @@ -19,9 +18,6 @@ jest.mock('standard-version', () => jest.fn()); jest.mock('standard-version/lib/lifecycles/changelog', () => jest.fn()); jest.mock('./utils/try-bump'); -// @todo get rid of this -_enableWip(); - describe('@jscutlery/semver:version', () => { const mockChangelog = changelog as jest.Mock; const mockExecFile = execFile as jest.MockedFunction; diff --git a/packages/semver/src/builders/version/version.ts b/packages/semver/src/builders/version/version.ts index 0133655f4..90cc030e4 100644 --- a/packages/semver/src/builders/version/version.ts +++ b/packages/semver/src/builders/version/version.ts @@ -9,13 +9,6 @@ import { } from './utils/changelog'; import { getPackageFiles, getProjectRoots } from './utils/workspace'; -// @todo get rid of this -let _isWip = process.env['JSCUTLERY_SEMVER_WIP'] === 'true'; - -export function _enableWip() { - _isWip = true; -} - export interface CommonVersionOptions { dryRun: boolean; newVersion: string; @@ -35,26 +28,24 @@ export function versionWorkspace({ } & CommonVersionOptions) { return concat( ...[ - _isWip - ? getProjectRoots(workspaceRoot).pipe( - switchMap((projectRoots) => - concat( - ...projectRoots - /* Don't update the workspace's changelog as it will be - * dealt with by `standardVersion`. */ - .filter((projectRoot) => projectRoot !== workspaceRoot) - .map((projectRoot) => - updateChangelog({ - dryRun: options.dryRun, - preset: options.preset, - projectRoot, - newVersion: options.newVersion, - }) - ) + getProjectRoots(workspaceRoot).pipe( + switchMap((projectRoots) => + concat( + ...projectRoots + /* Don't update the workspace's changelog as it will be + * dealt with by `standardVersion`. */ + .filter((projectRoot) => projectRoot !== workspaceRoot) + .map((projectRoot) => + updateChangelog({ + dryRun: options.dryRun, + preset: options.preset, + projectRoot, + newVersion: options.newVersion, + }) ) - ) ) - : [], + ) + ), getPackageFiles(workspaceRoot).pipe( switchMap((packageFiles) => _runStandardVersion({