Skip to content

test(@angular-devkit/build-webpack): remove unused createConsoleLogger usage#33171

Merged
alan-agius4 merged 2 commits into
angular:mainfrom
clydin:test/remove-createConsoleLogger-usage
May 13, 2026
Merged

test(@angular-devkit/build-webpack): remove unused createConsoleLogger usage#33171
alan-agius4 merged 2 commits into
angular:mainfrom
clydin:test/remove-createConsoleLogger-usage

Conversation

@clydin
Copy link
Copy Markdown
Member

@clydin clydin commented May 12, 2026

Removes the last usage of createConsoleLogger in the repository from the webpack builder tests. This makes the test consistent with others in the file and allows for potential removal of the function.

…r usage

Removes the last usage of `createConsoleLogger` in the repository from
the webpack builder tests. This makes the test consistent with others
in the file and allows for potential removal of the function.

Also removes an unused `BuildResult` import in the same file.
@clydin clydin added the target: minor This PR is targeted for the next minor release label May 12, 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 simplifies the Webpack builder test suite by removing unused imports, specifically createConsoleLogger and BuildResult, and streamlining the architect.scheduleTarget call within the 'works' test case. I have no feedback to provide as no review comments were submitted.

`createConsoleLogger` is deprecated. Use a custom logger implementation instead. This does not apply to application code.
@clydin clydin force-pushed the test/remove-createConsoleLogger-usage branch from 146238e to 77b383b Compare May 12, 2026 15:28
@clydin clydin added target: rc This PR is targeted for the next release-candidate target: major This PR is targeted for the next major release target: minor This PR is targeted for the next minor release action: review The PR is still awaiting reviews from at least one requested reviewer and removed target: minor This PR is targeted for the next minor release target: rc This PR is targeted for the next release-candidate target: major This PR is targeted for the next major release labels May 12, 2026
@clydin clydin requested review from alan-agius4 May 12, 2026 18:28
@alan-agius4 alan-agius4 added target: rc This PR is targeted for the next release-candidate and removed target: minor This PR is targeted for the next minor release labels May 12, 2026
@clydin clydin removed the request for review from alan-agius4 May 12, 2026 19:19
@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 May 12, 2026
@alan-agius4 alan-agius4 merged commit 015bbcb into angular:main May 13, 2026
41 checks passed
@alan-agius4
Copy link
Copy Markdown
Collaborator

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

@clydin clydin deleted the test/remove-createConsoleLogger-usage branch May 13, 2026 14:27
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: @angular-devkit/build-webpack area: @angular-devkit/core target: rc This PR is targeted for the next release-candidate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants