Skip to content

build: move from using rollup to esbuild for bundling zone.js #62775

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 2 commits into from
Closed

build: move from using rollup to esbuild for bundling zone.js #62775

wants to merge 2 commits into from

Conversation

josephperrott
Copy link
Member

No description provided.

@josephperrott josephperrott added area: build & ci Related the build and CI infrastructure of the project target: patch This PR is targeted for the next patch release labels Jul 23, 2025
@ngbot ngbot bot modified the milestone: Backlog Jul 23, 2025
@josephperrott josephperrott added target: minor This PR is targeted for the next minor release and removed target: patch This PR is targeted for the next patch release labels Jul 23, 2025
@josephperrott josephperrott force-pushed the migrate-zone-rollup branch 3 times, most recently from 188cc4d to 6628f1d Compare July 24, 2025 14:13
@josephperrott josephperrott marked this pull request as ready for review July 24, 2025 14:32
@josephperrott josephperrott added the action: merge The PR is ready for merge by the caretaker label Jul 24, 2025
@pullapprove pullapprove bot requested a review from AndrewKushnir July 24, 2025 14:32
@josephperrott josephperrott requested a review from devversion July 24, 2025 14:32
@crisbeto
Copy link
Member

Looks like this has conflicts.

@josephperrott
build: move from using rollup to esbuild for bundling zone.js
2acec74 
Use esbuild to bundle zone.js
@josephperrott
build: remove local zone.js build from usage in integration tests
5a80899 
Remove the local zone.js build from dependencies in the integration tests
@crisbeto
Copy link
Member

This PR was merged into the repository by commit b49c1f2.

The changes were merged into the following branches: main

@crisbeto crisbeto closed this in 7c06aca Jul 25, 2025
crisbeto pushed a commit that referenced this pull request Jul 25, 2025
@josephperrott @crisbeto
build: remove local zone.js build from usage in integration tests (#6…
b49c1f2 
…2775)

Remove the local zone.js build from dependencies in the integration tests

PR Close #62775
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@crisbeto crisbeto crisbeto approved these changes

@hardikgosai1 hardikgosai1 hardikgosai1 approved these changes

Assignees
No one assigned
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: minor This PR is targeted for the next minor release
Projects
None yet
Milestone
Backlog
Development

Successfully merging this pull request may close these issues.
3 participants
@josephperrott @crisbeto @hardikgosai1
pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy