Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update migrate-only requires to argument #24510

Closed
1 task done
bgotink opened this issue Jan 10, 2023 · 1 comment · Fixed by #24516
Closed
1 task done

Update migrate-only requires to argument #24510

bgotink opened this issue Jan 10, 2023 · 1 comment · Fixed by #24516
Labels
area: @angular/cli freq1: low Only reported by a handful of users who observe it rarely severity5: regression type: bug/fix

Comments

@bgotink
Copy link
Contributor

bgotink commented Jan 10, 2023

Command

update

Is this a regression?

  • Yes, this behavior used to work in the previous version

The previous version in which this bug was not present was

13.x

Description

Using ng update --migrate-only --from <version> <name> used to work:

$ NG_DISABLE_VERSION_CHECK=1 yarn dlx --package @angular/cli@13.x \
      ng update --migrate-only --from 14.0.0 @angular/cdk
Using package manager: 'yarn'
Collecting installed dependencies...
Found 74 dependencies.
** Executing migrations of package '@angular/cdk' **

❯ Updates the Angular CDK to v15.
...

The docs also point to this:

$ yarn run -TB ng update --help
ng update [packages..]

...
      --to              Version up to which to apply migrations. Only available with a single
                        package being updated, and only with 'migrate-only' option. Requires 'from'
                        to be specified. Default to the installed version detected.         [string]
...

Minimal Reproduction

$ ng update --migrate-only --from 14.0.0 @angular/cdk

Exception or Error

Error: Missing dependent arguments:
 from -> to

Your Environment

_                      _                 ____ _     ___
    / \   _ __   __ _ _   _| | __ _ _ __     / ___| |   |_ _|
   / △ \ | '_ \ / _` | | | | |/ _` | '__|   | |   | |    | |
  / ___ \| | | | (_| | |_| | | (_| | |      | |___| |___ | |
 /_/   \_\_| |_|\__, |\__,_|_|\__,_|_|       \____|_____|___|
                |___/
    

Angular CLI: 15.0.0
Node: 16.16.0
Package Manager: yarn 3.2.4
OS: darwin x64

Angular: 15.0.0
... cdk, cli, common, compiler, compiler-cli, core, elements
... forms, localize, platform-browser, platform-browser-dynamic
... router

Package                         Version
---------------------------------------------------------
@angular-devkit/architect       0.1500.0
@angular-devkit/build-angular   15.0.0
@angular-devkit/core            15.0.0
@angular-devkit/schematics      15.0.0
@schematics/angular             15.0.0
rxjs                            7.5.5
typescript                      4.8.2

Anything else relevant?

No response

@alan-agius4 alan-agius4 added type: bug/fix freq1: low Only reported by a handful of users who observe it rarely severity5: regression area: @angular/cli labels Jan 10, 2023
alan-agius4 added a commit to alan-agius4/angular-cli that referenced this issue Jan 11, 2023
…g `--from` in `ng update`

This change remove the requirment for the `to` option to be provided when using the `from` option in conjunction with `migrate-only`.

Closes angular#24510
alan-agius4 added a commit to alan-agius4/angular-cli that referenced this issue Jan 11, 2023
…g `--from` in `ng update`

This change remove the requirement for the `to` option to be provided when using the `from` option in conjunction with `migrate-only`.

Closes angular#24510
angular-robot bot pushed a commit that referenced this issue Jan 13, 2023
…g `--from` in `ng update`

This change remove the requirement for the `to` option to be provided when using the `from` option in conjunction with `migrate-only`.

Closes #24510
angular-robot bot pushed a commit that referenced this issue Jan 13, 2023
…g `--from` in `ng update`

This change remove the requirement for the `to` option to be provided when using the `from` option in conjunction with `migrate-only`.

Closes #24510

(cherry picked from commit 697df4f)
@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Feb 13, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area: @angular/cli freq1: low Only reported by a handful of users who observe it rarely severity5: regression type: bug/fix
Projects
None yet
2 participants