Skip to content

refactor: remove @angular-devkit/core logger and its dependencies from the e2e runner#32736

Merged
alan-agius4 merged 1 commit intoangular:mainfrom
alan-agius4:remove-unused-logger
Mar 10, 2026
Merged

refactor: remove @angular-devkit/core logger and its dependencies from the e2e runner#32736
alan-agius4 merged 1 commit intoangular:mainfrom
alan-agius4:remove-unused-logger

Conversation

@alan-agius4
Copy link
Collaborator

This logger was unused.

…rom the e2e runner

This logger was unused.
@alan-agius4 alan-agius4 requested a review from clydin March 10, 2026 13:50
@alan-agius4 alan-agius4 added the target: patch This PR is targeted for the next patch release label Mar 10, 2026
@alan-agius4 alan-agius4 marked this pull request as ready for review March 10, 2026 13:50
@alan-agius4 alan-agius4 added the action: review The PR is still awaiting reviews from at least one requested reviewer label Mar 10, 2026
@alan-agius4 alan-agius4 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 10, 2026
@alan-agius4 alan-agius4 merged commit 231f579 into angular:main Mar 10, 2026
92 of 95 checks passed
@alan-agius4 alan-agius4 deleted the remove-unused-logger branch March 10, 2026 14:18
@alan-agius4
Copy link
Collaborator Author

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

Copy link

@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 refactors the e2e runner by removing the unused @angular-devkit/core logger and its associated dependencies. The changes are clean and effectively remove the dead code from tests/e2e_runner.ts and the corresponding build configurations in tests/BUILD.bazel. The removal of an unused rollup plugin dependency is also a good cleanup. The changes are correct and improve code maintainability.

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 target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants