Skip to content

Use zone from npm through parts of the repo #61947

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

Use zone from npm through parts of the repo #61947

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
aa6f524 
Add zone.js as an npm based dependency
@josephperrott
refactor(devtools): use zone.js from npm instead of from the local build
347f1bc 
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…
7e3ecf1 
…gration tests

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

Use zone from npm for modules testing
@josephperrott josephperrott requested a review from devversion June 6, 2025 18:39
@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: minor This PR is targeted for the next minor release 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 approved these changes Jun 6, 2025
View reviewed changes
@AndrewKushnir
Copy link
Contributor

This PR was merged into the repository by commit 21fc93b.

The changes were merged into the following branches: main

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…
708e4f4 
…ild (#61947)

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

PR Close #61947
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…
50c7e13 
…gration tests (#61947)

Use zone from npm for integration tests

PR Close #61947
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…
21fc93b 
…les testing (#61947)

Use zone from npm for modules testing

PR Close #61947
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: 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 @AndrewKushnir @devversion