Skip to content

build: remove unused bazel build dependencies#3623

Merged
josephperrott merged 1 commit intoangular:mainfrom
xuhongxu96:main
Apr 24, 2026
Merged

build: remove unused bazel build dependencies#3623
josephperrott merged 1 commit intoangular:mainfrom
xuhongxu96:main

Conversation

@xuhongxu96
Copy link
Copy Markdown
Contributor

We've identified many redundant dependencies in bazel build scripts and refactored them, as part of a research project on dependency optimization.

Feel free to leave feedback, suggest improvements, or directly edit this PR.

Thanks for your support!

@angular-robot angular-robot Bot added the area: build & ci Related the build and CI infrastructure of the project label Apr 23, 2026
Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request removes several internal dependencies from BUILD.bazel files across the apps/code-of-conduct and ng-dev directories, including testing utilities, commit message validation, and release-related packages. I have no feedback to provide.

Copy link
Copy Markdown
Member

@josephperrott josephperrott left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@josephperrott josephperrott added the action: merge The PR is ready for merge by the caretaker label Apr 24, 2026
@josephperrott josephperrott merged commit c4d0c37 into angular:main Apr 24, 2026
13 checks passed
@josephperrott
Copy link
Copy Markdown
Member

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

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants