Skip to content

Conversation

@filipesilva
Copy link
Contributor

We only use it for a few things but have a strict peerdep.

This strictness causes errors when updating the CLI from 7.x to 8.x projects:

Package "@angular/compiler-cli" has an incompatible peer dependency to "typescript" (requires ">=3.1.1 <3.2", would install "3.4.5").

build-angular did not have a peerdep in 7.x so this never was a problem.

This PR relaxes the peerdep to ">=3.1 < 3.5", which covers the 3.1 used in initial CLI 7.0 projects.

@filipesilva filipesilva requested a review from clydin May 3, 2019 10:01
@filipesilva filipesilva added the target: patch This PR is targeted for the next patch release label May 3, 2019
We only use it for a few things but have a strict peerdep.

This strictness causes errors when updating the CLI from 7.x to 8.x projects:
```
Package "@angular/compiler-cli" has an incompatible peer dependency to "typescript" (requires ">=3.1.1 <3.2", would install "3.4.5").
```
`build-angular` did not have a peerdep in 7.x so this never was a problem.

This PR relaxes the peerdep to `">=3.1 < 3.5"`, which covers the 3.1 used in initial CLI 7.0 projects.
@filipesilva filipesilva force-pushed the build-angular-ts-peerdep branch from 908fbe4 to 99c1c69 Compare May 3, 2019 10:03
@filipesilva filipesilva mentioned this pull request May 7, 2019
@alexeagle alexeagle merged commit a41c185 into angular:master May 7, 2019
@filipesilva filipesilva deleted the build-angular-ts-peerdep branch May 7, 2019 22:24
@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 Sep 13, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants