Skip to content

[20.0.x] Use zone from npm through parts of the repo #61948

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 4 commits into from
Closed

[20.0.x] Use zone from npm through parts of the repo #61948

wants to merge 4 commits into from

Conversation

josephperrott
Copy link
Member

No description provided.

@josephperrott
build: add zone.js as an npm based dependency
38e902e 
Add zone.js as an npm based dependency
@josephperrott
refactor(devtools): use zone.js from npm instead of from the local build
641cad9 
Rely on zone.js from the npm managed version instead of the locally built copy
@josephperrott
test: use zone from npm instead of the locally built package for inte…
cf61496 
…gration tests

Use zone from npm for integration tests
@josephperrott
test: use zone from npm instead of the locally built package for modu…
9f11e22 
…les testing

Use zone from npm for modules testing
@josephperrott josephperrott requested a review from devversion June 6, 2025 18:56
@josephperrott josephperrott added action: merge The PR is ready for merge by the caretaker area: build & ci Related the build and CI infrastructure of the project target: patch This PR is targeted for the next patch release PullApprove: disable labels Jun 6, 2025
@ngbot ngbot bot added this to the Backlog milestone Jun 6, 2025
@josephperrott josephperrott marked this pull request as ready for review June 6, 2025 19:18
@devversion devversion changed the title Use zone from npm through parts of the repo [20.0.x] Use zone from npm through parts of the repo Jun 6, 2025
@AndrewKushnir
Copy link
Contributor

This PR was merged into the repository by commit 74b2aa6.

The changes were merged into the following branches: 20.0.x

AndrewKushnir pushed a commit that referenced this pull request Jun 6, 2025
@josephperrott @AndrewKushnir
build: add zone.js as an npm based dependency (#61948)
d3c2956 
Add zone.js as an npm based dependency

PR Close #61948
AndrewKushnir pushed a commit that referenced this pull request Jun 6, 2025
@josephperrott @AndrewKushnir
refactor(devtools): use zone.js from npm instead of from the local bu…
10b9bce 
…ild (#61948)

Rely on zone.js from the npm managed version instead of the locally built copy

PR Close #61948
AndrewKushnir pushed a commit that referenced this pull request Jun 6, 2025
@josephperrott @AndrewKushnir
test: use zone from npm instead of the locally built package for inte…
6d7d190 
…gration tests (#61948)

Use zone from npm for integration tests

PR Close #61948
AndrewKushnir pushed a commit that referenced this pull request Jun 6, 2025
@josephperrott @AndrewKushnir
test: use zone from npm instead of the locally built package for modu…
74b2aa6 
…les testing (#61948)

Use zone from npm for modules testing

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

@devversion devversion devversion 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 area: devtools PullApprove: disable target: patch This PR is targeted for the next patch release
Projects
None yet
Milestone
Backlog
Development

Successfully merging this pull request may close these issues.
3 participants
@josephperrott @AndrewKushnir @devversion