WIP

packages/turbo-workspaces/src/cli.ts

@@ -25,6 +25,11 @@ workspacesCli
     "Do not run a package manager install after conversion",
     false
   )
+  .option(
+    "--ignore-unchanged-package-manager",
+    "Always exit with a successful status (exit code 0)",
+    false
+  )
   .option("--dry", "Dry run (no changes are made to files)", false)
   .option(
     "--force",

packages/turbo-workspaces/src/convert.ts

@@ -36,19 +36,21 @@ export async function convertProject({
     `Converting project from ${project.packageManager} to ${convertTo.name}.`
   );
 
-  if (project.packageManager === convertTo.name) {
-    throw new ConvertError("You are already using this package manager", {
-      type: "package_manager-already_in_use",
-    });
-  }
+  if (!options?.ignoreUnchangedPackageManager) {
+    if (project.packageManager === convertTo.name) {
+      throw new ConvertError("You are already using this package manager", {
+        type: "package_manager-already_in_use",
+      });
+    }
 
-  if (!convertTo.version) {
-    throw new ConvertError(
-      `${convertTo.name} is not installed, or could not be located`,
-      {
-        type: "package_manager-could_not_be_found",
-      }
-    );
+    if (!convertTo.version) {
+      throw new ConvertError(
+        `${convertTo.name} is not installed, or could not be located`,
+        {
+          type: "package_manager-could_not_be_found",
+        }
+      );
+    }
   }
 
   // this cast is safe since we've just verified that the version exists above

packages/turbo-workspaces/src/types.ts

@@ -92,6 +92,7 @@ export interface Options {
   dry?: boolean;
   skipInstall?: boolean;
   interactive?: boolean;
+  ignoreUnchangedPackageManager?: boolean;
 }
 
 export interface PackageManagerInstallDetails {

turborepo-tests/example-basic-npm/package.json

@@ -6,6 +6,5 @@
   "dependencies": {
     "turborepo-tests-helpers": "workspace:*",
     "turborepo-examples": "workspace:*"
-  },
-  "packageManager": "npm"
+  }
 }

turborepo-tests/helpers/package.json

@@ -1,3 +1,9 @@
 {
-  "name": "turborepo-tests-helpers"
+  "name": "turborepo-tests-helpers",
+  "scripts": {
+    "test": ""
+  },
+  "dependencies": {
+    "@turbo/workspaces": "workspace:*"
+  }
 }

turborepo-tests/helpers/setup_example_test.sh

@@ -21,13 +21,15 @@ fi
 
 echo "node --version: $(node --version)"
 
+echo "$PWD"
+
 # Use the right command for each package manager
 if [ "$package_manager" == "npm" ]; then
-  package_manager_command="bash -c 'npx @turbo/workspaces convert . npm || true' && npm install"
+  package_manager_command="node ../../../packages/turbo-workspaces/dist/cli.js convert . npm --ignore-unchanged-package-manager || true && npm install"
 elif [ "$package_manager" == "pnpm" ]; then
-  package_manager_command="bash -c 'npx @turbo/workspaces convert . pnpm || true' && pnpm install"
+  package_manager_command="node ../../../packages/turbo-workspaces/dist/cli.js convert . pnpm --ignore-unchanged-package-manager || true && pnpm install"
 elif [ "$package_manager" == "yarn" ]; then
-  package_manager_command="bash -c 'npx @turbo/workspaces convert . yarn || true' && yarn"
+  package_manager_command="node ../../../packages/turbo-workspaces/dist/cli.js convert . yarn --ignore-unchanged-package-manager || true && yarn"
 fi
 
 # All examples implement these two tasks
@@ -39,18 +41,18 @@ mkdir -p ../../examples-tests-tmp
 cd ../../examples-tests-tmp
 
 # Start up a fresh directory for the test
-rm -rf "$example_path" || true
+rm -rf "$example_path-$package_manager" || true
 rsync -avq \
   --exclude='node_modules' \
   --exclude="dist" \
   --exclude=".turbo" \
   --exclude=".expo" \
   --exclude=".cache" \
   --exclude=".next" \
-  "../examples/$example_path" "."
+  "../examples/$example_path" "$example_path-$package_manager"
 
-cd "$example_path"
-"../../turborepo-tests/helpers/setup_git.sh" .
+cd "$example_path-$package_manager/$example_path"
+"../../../turborepo-tests/helpers/setup_git.sh" .
 
 # Make /tmp dir for writing dump logs
 mkdir -p ./tmp

turborepo-tests/helpers/turbo.json

@@ -3,6 +3,9 @@
   "tasks": {
     // This needs to exist because upstream packages depend on it and
     // it's not defined at the root level.
-    "topo": {}
+    "topo": {},
+    "test": {
+      "dependsOn": ["^build"]
+    }
   }
 }