fix(core): migrate using yarn when nxJson has another package manager configured

[johnhwhite]

nx test nx now tests the detectPackageManager function

Current Behavior

nx migrate --run-migrations uses yarn despite nx.json configuration.

Expected Behavior

nx migrate --run-migrations should respect nx.json configuration.

Related Issue(s)

Fixes #16836

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
nx-dev	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	May 7, 2023 3:54pm

[nx-cloud]

☁️ Nx Cloud Report

CI is running/has finished running commands for commit f0e2e24. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this branch

---

✅ Successfully ran 8 targets

• nx affected -t e2e-macos --parallel=1 --base= --head=
• nx affected --target=e2e --base=832fa71488c9afb48fa255b76808385977bedd35 --head=f0e2e24a5e72cdd280b83707be68fa767f74bf37 --parallel=1
• nx run-many -t check-imports check-commit check-lock-files depcheck check-codeowners --parallel=1 --no-dte
• nx affected --target=lint --base=832fa71488c9afb48fa255b76808385977bedd35 --head=f0e2e24a5e72cdd280b83707be68fa767f74bf37 --parallel=3
• nx affected --target=build --base=832fa71488c9afb48fa255b76808385977bedd35 --head=f0e2e24a5e72cdd280b83707be68fa767f74bf37 --parallel=3
• nx affected --target=test --base=832fa71488c9afb48fa255b76808385977bedd35 --head=f0e2e24a5e72cdd280b83707be68fa767f74bf37 --parallel=1
• nx-cloud record -- nx format:check --base=832fa71488c9afb48fa255b76808385977bedd35 --head=f0e2e24a5e72cdd280b83707be68fa767f74bf37
• nx documentation --no-dte

---

Sent with 💌 from NxCloud.

[github-actions]

This pull request has already been merged/closed. If you experience issues related to these changes, please open a new issue referencing this pull request.