Skip to content

Conversation

@clydin
Copy link
Member

@clydin clydin commented Dec 6, 2025

This commit refactors the ng update command by extracting functions from packages/angular/cli/src/commands/update/cli.ts into more focused utility modules:

  • packages/angular/cli/src/commands/update/utilities/git.ts: Encapsulates Git-related operations such as checking for a clean repository, creating commits, and retrieving commit hashes.
  • packages/angular/cli/src/commands/update/utilities/cli-version.ts: Handles CLI version compatibility checks, temporary CLI installations, and package manager force options.
  • packages/angular/cli/src/commands/update/utilities/constants.ts: Stores shared constants like ANGULAR_PACKAGES_REGEXP.

This commit refactors the `ng update` command by extracting
functions from `packages/angular/cli/src/commands/update/cli.ts`
into more focused utility modules:

- `packages/angular/cli/src/commands/update/utilities/git.ts`:
  Encapsulates Git-related operations such as checking for a clean
  repository, creating commits, and retrieving commit hashes.
- `packages/angular/cli/src/commands/update/utilities/cli-version.ts`:
  Handles CLI version compatibility checks, temporary CLI installations,
  and package manager force options.
- `packages/angular/cli/src/commands/update/utilities/constants.ts`:
  Stores shared constants like `ANGULAR_PACKAGES_REGEXP`.
@clydin clydin added the target: minor This PR is targeted for the next minor release label Dec 6, 2025
@clydin clydin added the action: merge The PR is ready for merge by the caretaker label Dec 6, 2025
@clydin clydin merged commit 9d6891f into angular:main Dec 6, 2025
37 checks passed
@clydin
Copy link
Member Author

clydin commented Dec 6, 2025

This PR was merged into the repository. The changes were merged into the following branches:

@clydin clydin deleted the cli/update-cleanup-1 branch December 6, 2025 02:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

action: merge The PR is ready for merge by the caretaker area: @angular/cli target: minor This PR is targeted for the next minor release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants