forked from microsoft/TypeScript
-
Notifications
You must be signed in to change notification settings - Fork 0
Move compiler-debug into Debug namespace, which allows the compiler to be tree shaken #33
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
Closed
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
79f21b3 to
ce2b59b
Compare
c71f8ba to
b88df81
Compare
ce2b59b to
b124ac1
Compare
5c81d1a to
3d4c9f0
Compare
435ba70 to
49047e7
Compare
3d4c9f0 to
2d7af0f
Compare
49047e7 to
a96025d
Compare
2d7af0f to
41e57d6
Compare
a96025d to
26681a1
Compare
41e57d6 to
95bb6e2
Compare
26681a1 to
535e94d
Compare
95bb6e2 to
f782b2c
Compare
535e94d to
b0f7ebf
Compare
f782b2c to
e5fd341
Compare
b0f7ebf to
9a8a98e
Compare
e5fd341 to
26ba28d
Compare
9a8a98e to
c08a758
Compare
26ba28d to
9c72255
Compare
c08a758 to
3e53449
Compare
9c72255 to
229d522
Compare
3e53449 to
725b6c7
Compare
229d522 to
9e92519
Compare
725b6c7 to
dc69cb3
Compare
9e92519 to
c2e8900
Compare
dc69cb3 to
bb59776
Compare
f9f8e9e to
a7d2107
Compare
e1ee5ae to
384ab90
Compare
a7d2107 to
1f616d1
Compare
384ab90 to
9f324c5
Compare
1f616d1 to
393d0de
Compare
9f324c5 to
67d85a4
Compare
393d0de to
540a735
Compare
67d85a4 to
7360152
Compare
540a735 to
abb36fd
Compare
7360152 to
6aa7f3e
Compare
abb36fd to
9f30548
Compare
6aa7f3e to
47028cb
Compare
9f30548 to
935be03
Compare
47028cb to
c2c6386
Compare
935be03 to
51893a6
Compare
c2c6386 to
a524243
Compare
51893a6 to
7e5b706
Compare
a524243 to
08fe163
Compare
7e5b706 to
3fa05c2
Compare
08fe163 to
654a6e3
Compare
3fa05c2 to
77eb926
Compare
654a6e3 to
175fc44
Compare
77eb926 to
df9c9f9
Compare
175fc44 to
a6091c0
Compare
df9c9f9 to
cf0b856
Compare
…o be tree shaken This debug code was added quite a while ago, constructed such that we wouldn't have to ship this code to our users. However, this is the sole place in the compiler project where the ts namespace "escapes" the bundle. By moving this debug code into the compiler itself, we no longer have any references to the ts namespace itself for our bundles that don't export anything (tsc, typingsInstaller). This lets bundlers tree shake the compiler, reducing the size of our output by _5.7 MB_ (a ridiculous improvement for _adding_ code).
a6091c0 to
32515e4
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This debug code was added quite a while ago, constructed such that we
wouldn't have to ship this code to our users.
However, this is the sole place in the compiler project where the ts
namespace "escapes" the bundle. By moving this debug code into the
compiler itself, we no longer have any references to the ts namespace
itself for our bundles that don't export anything (tsc,
typingsInstaller). This lets bundlers tree shake the compiler, reducing
the size of our output by 5.7 MB (a ridiculous improvement for
adding code).
Please do not comment on this PR. Depending on how this set of PRs evolves, this PR's contents may change entirely based on the order of commits.
This PR is a part of a stack: