Skip to content

Commit 19cb354

Browse files
alan-agius4mgechev
authored andcommitted
refactor: don't drop zone.js from polyfills
We don't know yet if we are going to separate zone.js in a separate file. Hence, we should remove this for the time being and include it only when we remove zone.js from the main polyfills files
1 parent 87b0406 commit 19cb354

File tree

2 files changed

+2
-3
lines changed

2 files changed

+2
-3
lines changed

packages/schematics/angular/migrations/update-8/drop-es6-polyfills.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,6 @@ const toDrop: {[importName: string]: true} = {
2222
'core-js/es6/array': true,
2323
'core-js/es6/regexp': true,
2424
'core-js/es6/map': true,
25-
'zone.js/dist/zone': true,
2625
'core-js/es6/set': true,
2726
};
2827

packages/schematics/angular/migrations/update-8/drop-es6-polyfills_spec.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -80,8 +80,8 @@ import 'zone.js/dist/zone'; // Included with Angular CLI.
8080
const polyfills = tree.readContent(polyfillsPath);
8181
expect(polyfills).not.toContain('core-js/es6/symbol');
8282
expect(polyfills).not.toContain('core-js/es6/set');
83-
expect(polyfills).not.toContain('zone.js');
84-
expect(polyfills).not.toContain('Zone');
83+
expect(polyfills).toContain('zone.js');
84+
expect(polyfills).toContain('Zone');
8585

8686
// We don't want to drop this commented import comments
8787
expect(polyfills).toContain('core-js/es6/reflect');

0 commit comments

Comments
 (0)