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
Removing const / let in favor of var, for Chrome 45. See #2098. #2162
Conversation
Overall size impact on
|
file | event | diff | master |
issue-2098 |
---|---|---|---|---|
extras/amd.js | changed | +6 | 5,279 | 5,285 |
extras/amd.min.js | changed | -67 | 1,706 | 1,639 |
extras/global.js | changed | -14 | 2,848 | 2,834 |
extras/global.min.js | changed | -24 | 814 | 790 |
extras/module-types.js | changed | -24 | 3,669 | 3,645 |
extras/module-types.min.js | changed | -14 | 1,601 | 1,587 |
extras/named-exports.js | changed | +10 | 2,266 | 2,276 |
extras/named-exports.min.js | changed | -24 | 675 | 651 |
extras/named-register.js | changed | -24 | 2,327 | 2,303 |
extras/named-register.min.js | changed | -22 | 883 | 861 |
extras/transform.js | changed | -6 | 1,163 | 1,157 |
extras/transform.min.js | changed | -4 | 594 | 590 |
extras/use-default.js | changed | -4 | 513 | 509 |
extras/use-default.min.js | changed | -2 | 253 | 251 |
s.js | changed | +34 | 19,306 | 19,340 |
s.min.js | changed | -119 | 6,069 | 5,950 |
system-node.cjs | changed | -142 | 1,397,449 | 1,397,307 |
system.js | changed | -87 | 29,489 | 29,402 |
system.min.js | changed | -182 | 9,956 | 9,774 |
Overall size impact: -709 bytes.
Cache impact: 19 files content changed, invalidating a total of 1,486,860 bytes.
src/features/node-fetch.js
Outdated
@@ -6,12 +6,12 @@ import { fileURLToPath } from 'url'; | |||
import rimraf from 'rimraf'; |
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.
No need to do it for this file.
src/system-node.js
Outdated
@@ -7,7 +7,7 @@ import './extras/module-types.js'; | |||
import './features/node-fetch.js'; | |||
import { BASE_URL } from './common.js'; | |||
|
|||
export const System = global.System; | |||
export var System = global.System; |
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.
Or for this one.
The tests are failing for this - it appears the named-exports extra was relying on the semantics of const/let in a way that doesn't work with var. I am working on updating it. |
The tests are failing on the master branch, too. I will submit a subsequent PR to fix the tests. |
See #2098