Skip to content

refactor(@angular/build): remove direct usage of undici dependency for SSR#28010

Merged
alan-agius4 merged 1 commit intoangular:mainfrom
clydin:application/remove-undici-dep
Jul 8, 2024
Merged

refactor(@angular/build): remove direct usage of undici dependency for SSR#28010
alan-agius4 merged 1 commit intoangular:mainfrom
clydin:application/remove-undici-dep

Conversation

@clydin
Copy link
Copy Markdown
Member

@clydin clydin commented Jul 8, 2024

The Node.js types (@types/node) now contains the information for the Response class and the global fetch function. The Response object is also globally accessible in all versions of Node.js supported by the Angular CLI. This removes the need to depend directly on the undici package. The undici package is still used for unit-testing and remains as a development dependency.

…r SSR

The Node.js types (`@types/node`) now contains the information for the `Response`
class and the global `fetch` function. The `Response` object is also globally
accessible in all versions of Node.js supported by the Angular CLI. This removes
the need to depend directly on the `undici` package.
The `undici` package is still used for unit-testing and remains as a development
dependency.
@clydin clydin force-pushed the application/remove-undici-dep branch from 90c9e2c to 56b77e5 Compare July 8, 2024 15:17
@clydin clydin marked this pull request as ready for review July 8, 2024 15:34
@clydin clydin requested a review from alan-agius4 July 8, 2024 15:34
@clydin clydin added action: review The PR is still awaiting reviews from at least one requested reviewer target: minor This PR is targeted for the next minor release labels Jul 8, 2024
@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 Jul 8, 2024
@alan-agius4 alan-agius4 merged commit 4286bb0 into angular:main Jul 8, 2024
@clydin clydin deleted the application/remove-undici-dep branch July 8, 2024 17:11
@angular-automatic-lock-bot
Copy link
Copy Markdown

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Aug 8, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: merge The PR is ready for merge by the caretaker target: minor This PR is targeted for the next minor release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants