build: move TypeScript and Angular Compiler CLI to dependencies - #32977
Conversation
There was a problem hiding this comment.
Code Review
This pull request moves @angular/compiler-cli and typescript from devDependencies to dependencies in package.json, with corresponding updates to the pnpm-lock.yaml file. Feedback indicates that @angular/compiler should also be moved to the dependencies section to satisfy the peer dependency requirements of @angular/compiler-cli and prevent potential installation failures in production environments.
This is needed by rules_angular as otherwise these two packages are not present in the `npm_package_store_infos` using in the symlink package.
c17d0b9 to
6371e2f
Compare
|
This PR was merged into the repository. The changes were merged into the following branches:
|
|
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
This is needed by rules_angular as otherwise these two packages are not present in the
npm_package_store_infosusing in the symlink package.See: https://github.com/alan-agius4/dev-infra/blob/6cd21db3992d23e5a8009d5379723a487214e092/bazel/rules/rules_angular/src/private/symlink_package.bzl#L24