Skip to content

Upgrade Rush to 5.175.1, pnpm to 10.34.1, and remove npm dependency#5189

Merged
timotheeguerin merged 9 commits into
mainfrom
timotheeguerin/upgrade-rush-pnpm
Jun 9, 2026
Merged

Upgrade Rush to 5.175.1, pnpm to 10.34.1, and remove npm dependency#5189
timotheeguerin merged 9 commits into
mainfrom
timotheeguerin/upgrade-rush-pnpm

Conversation

@timotheeguerin

Copy link
Copy Markdown
Member

Changes

  • Rush: 5.153.2 → 5.175.1
  • pnpm: 8.10.5 → 10.34.1
  • Removed unused npm dependency from @azure-tools/extension
    • Deleted src/npm.ts (dead code — npm package manager was never selected)
    • Simplified ExtensionManager.Create to always use Yarn
    • Removed PackageManagerType type
  • Added globalOnlyBuiltDependencies in pnpm-config.json (required by pnpm 10+ security model)

- Upgrade Rush from 5.153.2 to 5.175.1
- Upgrade pnpm from 8.10.5 to 10.34.1
- Remove unused npm dependency from @azure-tools/extension
- Delete src/npm.ts and simplify ExtensionManager to always use Yarn
- Add globalOnlyBuiltDependencies for pnpm 10 security model
@timotheeguerin timotheeguerin marked this pull request as ready for review June 9, 2026 17:23
@timotheeguerin timotheeguerin enabled auto-merge (squash) June 9, 2026 17:23
@azure-pipelines

Copy link
Copy Markdown

You may test this build by running autorest --reset and then either:


Add the following CLI flags
Pacakge Flag Description
@autorest/core --version:https://tinyurl.com/2xqsgxdx For changes to autorest core.
@autorest/modelerfour --use:https://tinyurl.com/2947zgja For changes to modelerfour.

Or with all

autorest --version:https://tinyurl.com/2xqsgxdx --use:https://tinyurl.com/2947zgja

or use the following in your autorest configuration:
# For changes to autorest core
version: "https://tinyurl.com/2xqsgxdx"

# For changes to modelerfour
use-extension:
  "@autorest/modelerfour": "https://tinyurl.com/2947zgja"

If this build is good for you, give this comment a thumbs up. (👍) And you should run `autorest --reset` again once you're finished testing to remove it.

@azure-pipelines

Copy link
Copy Markdown

You may test this build by running autorest --reset and then either:


Add the following CLI flags
Pacakge Flag Description
@autorest/core --version:https://tinyurl.com/29mhe9by For changes to autorest core.
@autorest/modelerfour --use:https://tinyurl.com/22kaffc9 For changes to modelerfour.

Or with all

autorest --version:https://tinyurl.com/29mhe9by --use:https://tinyurl.com/22kaffc9

or use the following in your autorest configuration:
# For changes to autorest core
version: "https://tinyurl.com/29mhe9by"

# For changes to modelerfour
use-extension:
  "@autorest/modelerfour": "https://tinyurl.com/22kaffc9"

If this build is good for you, give this comment a thumbs up. (👍) And you should run `autorest --reset` again once you're finished testing to remove it.

@timotheeguerin timotheeguerin force-pushed the timotheeguerin/upgrade-rush-pnpm branch from a2844cc to 9bea079 Compare June 9, 2026 18:26
@azure-pipelines

Copy link
Copy Markdown

You may test this build by running autorest --reset and then either:


Add the following CLI flags
Pacakge Flag Description
@autorest/core --version:https://tinyurl.com/2b69ucon For changes to autorest core.
@autorest/modelerfour --use:https://tinyurl.com/2cxoarzo For changes to modelerfour.

Or with all

autorest --version:https://tinyurl.com/2b69ucon --use:https://tinyurl.com/2cxoarzo

or use the following in your autorest configuration:
# For changes to autorest core
version: "https://tinyurl.com/2b69ucon"

# For changes to modelerfour
use-extension:
  "@autorest/modelerfour": "https://tinyurl.com/2cxoarzo"

If this build is good for you, give this comment a thumbs up. (👍) And you should run `autorest --reset` again once you're finished testing to remove it.

npm@6 is incompatible with Node 22 and is no longer needed since
Rush uses pnpm directly for packaging.
@azure-pipelines

Copy link
Copy Markdown

You may test this build by running autorest --reset and then either:


Add the following CLI flags
Pacakge Flag Description
@autorest/core --version:https://tinyurl.com/239tg8rn For changes to autorest core.
@autorest/modelerfour --use:https://tinyurl.com/2y79w4q4 For changes to modelerfour.

Or with all

autorest --version:https://tinyurl.com/239tg8rn --use:https://tinyurl.com/2y79w4q4

or use the following in your autorest configuration:
# For changes to autorest core
version: "https://tinyurl.com/239tg8rn"

# For changes to modelerfour
use-extension:
  "@autorest/modelerfour": "https://tinyurl.com/2y79w4q4"

If this build is good for you, give this comment a thumbs up. (👍) And you should run `autorest --reset` again once you're finished testing to remove it.

@timotheeguerin timotheeguerin merged commit fc5918b into main Jun 9, 2026
11 checks passed
@timotheeguerin timotheeguerin deleted the timotheeguerin/upgrade-rush-pnpm branch June 9, 2026 20:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants