fix(core): run migrations ordered by their target version

[leosvelperez]

Current Behavior

Most of the time, all migrations of a given plugin are run ordered by their target version. This can be problematic sometimes when a migration of a plugin checks stuff from another plugin.

Such an issue occurred and is explained in #20407 (comment):

• the update-tsconfig-spec-jest migration from the @nx/angular package targets the version 15.9.0-beta.3
• that migration checks targets with the @nrwl/jest:jest executor to perform an update to the tsconfig.spec.json
• the update-16-0-0-add-nx-packages migration from the @nx/jest package targets the version 16.0.0-beta.1
• that migration renames and replaces all occurrences of @nrwl/jest with @nx/jest
• the @nx/jest package migrations run before the @nx/angular package migrations

Note how @nrwl/jest:jest will be replaced @nx/jest:jest before running the update-tsconfig-spec-jest migration from the @nx/angular package, even though that migration was written/released and targets a lower version of Nx. That migration shouldn't know about the @nx/* scoped packages because it was written for a version where the rescoping hadn't occurred yet.

Expected Behavior

Migrations should be run ordered by their target version regardless of the plugin they are for.

Note: the 15-7-0-split-configuration-into-project-json-files migration from @nx/workspace is a special case that's meant to run first.

Related Issue(s)

Fixes #

[vercel]

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

1 Ignored Deployment

Name	Status	Preview	Updated (UTC)
nx-dev	⬜️ Ignored (Inspect)	Visit Preview	Feb 13, 2024 11:28am

[nx-cloud]

☁️ Nx Cloud Report

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

📂 See all runs for this CI Pipeline Execution

---

✅ Successfully ran 6 targets

• nx affected --target=e2e --base=7d2cb377842e2db2dff4c86f3e5634aba7c522da --head=ee779a16a3820d5759c86455d35d8f27933cf152 --parallel=1
• nx affected --targets=lint,test,build --base=7d2cb377842e2db2dff4c86f3e5634aba7c522da --head=ee779a16a3820d5759c86455d35d8f27933cf152 --parallel=3
• nx run-many -t check-imports check-commit check-lock-files check-codeowners documentation --parallel=1 --no-dte
• nx-cloud record -- nx format:check --base=7d2cb377842e2db2dff4c86f3e5634aba7c522da --head=ee779a16a3820d5759c86455d35d8f27933cf152
• nx documentation --no-dte
• nx affected -t e2e-macos --parallel=1 --base=7d2cb377842e2db2dff4c86f3e5634aba7c522da --head=ee779a16a3820d5759c86455d35d8f27933cf152

---

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.