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.
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
feat(zone.js): remove legacy files and access to deep imports #51752
feat(zone.js): remove legacy files and access to deep imports #51752
Conversation
This commit removes access to deep imports and `zone-testing-bundle` and `zone-testing-node-bundle` This commit removed access to deep and legacy `dist` imports. `zone-testing-bundle` and `zone-testing-node-bundle` are also no longer generated. BREAKING CHANGE: Deep and legacy `dist/` imports like `zone.js/bundles/zone-testing.js` and `zone.js/dist/zone` are no longer allowed. `zone-testing-bundle` and `zone-testing-node-bundle` are also no longer part of the package. The proper way to import `zone.js` and `zone.js/testing` is: ```js import 'zone.js'; import 'zone.js/testing'; ```
…839c18c7f6db81` This is needed to consume a zone.js related import fix. angular/dev-infra#1383
This commits add missing `override` modifier to `HostBindingCompilationJob` class members.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
This PR was merged into the repository by commit 70e88f8. |
…839c18c7f6db81` (#51752) This is needed to consume a zone.js related import fix. angular/dev-infra#1383 PR Close #51752
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
…r#51752) This commit removes access to deep imports and `zone-testing-bundle` and `zone-testing-node-bundle` This commit removed access to deep and legacy `dist` imports. `zone-testing-bundle` and `zone-testing-node-bundle` are also no longer generated. BREAKING CHANGE: Deep and legacy `dist/` imports like `zone.js/bundles/zone-testing.js` and `zone.js/dist/zone` are no longer allowed. `zone-testing-bundle` and `zone-testing-node-bundle` are also no longer part of the package. The proper way to import `zone.js` and `zone.js/testing` is: ```js import 'zone.js'; import 'zone.js/testing'; ``` PR Close angular#51752
…839c18c7f6db81` (angular#51752) This is needed to consume a zone.js related import fix. angular/dev-infra#1383 PR Close angular#51752
…ngular#51752) This commits add missing `override` modifier to `HostBindingCompilationJob` class members. PR Close angular#51752
This commit removes access to deep imports and
zone-testing-bundle
andzone-testing-node-bundle
This commit removed access to deep and legacy
dist
imports.zone-testing-bundle
andzone-testing-node-bundle
are also no longer generated.BREAKING CHANGE:
Deep and legacy
dist/
imports likezone.js/bundles/zone-testing.js
andzone.js/dist/zone
are no longer allowed.zone-testing-bundle
andzone-testing-node-bundle
are also no longer part of the package.The proper way to import
zone.js
andzone.js/testing
is: