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

build: move @angular/build specific dependencies out of root #29870

Merged
merged 1 commit into from
Mar 18, 2025

Conversation

clydin
Copy link
Member

@clydin clydin commented Mar 18, 2025

With the migration to rules_js, package specific dependencies now only need to be referenced with the source package.json for each specific package. Most of the @angular/build specific dependencies have now been moved. This is not exhaustive and further changes will continue to move additional dependencies.

@clydin clydin added the target: major This PR is targeted for the next major release label Mar 18, 2025
@angular-robot angular-robot bot added the area: build & ci Related the build and CI infrastructure of the project label Mar 18, 2025
@clydin clydin force-pushed the build/cleanup-core-deps branch from d88bc0a to 3c575b2 Compare March 18, 2025 14:00
With the migration to `rules_js`, package specific dependencies now only
need to be referenced with the source `package.json` for each specific
package. Most of the `@angular/build` specific dependencies have now been
moved. This is not exhaustive and further changes will continue to move
additional dependencies.
@clydin clydin force-pushed the build/cleanup-core-deps branch from 3c575b2 to 766593c Compare March 18, 2025 14:26
@clydin clydin marked this pull request as ready for review March 18, 2025 15:03
@clydin clydin requested a review from devversion March 18, 2025 15:03
@clydin clydin added the action: review The PR is still awaiting reviews from at least one requested reviewer label Mar 18, 2025
@clydin clydin added action: merge The PR is ready for merge by the caretaker and removed action: review The PR is still awaiting reviews from at least one requested reviewer labels Mar 18, 2025
@clydin clydin merged commit b66d36b into angular:main Mar 18, 2025
32 checks passed
@clydin clydin deleted the build/cleanup-core-deps branch March 18, 2025 17:01
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: build & ci Related the build and CI infrastructure of the project target: major This PR is targeted for the next major release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants