-
-
Notifications
You must be signed in to change notification settings - Fork 433
Enhancement: Implement Tsickle for Google Closure Compiler #224
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
Comments
Would love to see this! Need any help? |
If you'd like to have a crack at implementation then go for it! We gratefully accept PRs |
Did you have any preferences on the configuration for this? Like just a boolean flag to turn tsickle on or off from the user's point of view? |
That sounds like a good starting point. Essentially anything can be refined at any point though - have a play and see what you come up with would be my advice. |
Sounds good, I'll post back when I come up with something. |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
1 similar comment
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
Closing as stale. Please reopen if you'd like to work on this further. |
Currently there is an angular project which I think could benefit all Typescript users.
As we all know, Google closure compiler is still the most efficient minifier, however requires annotations for it to be used in ADVANCED mode.
https://github.com/angular/tsickle
This project is using a baked tsc that automagically converts decorators and types to annotations that allow for GCC to understand.
Since Webpack2 is implementing LoaderOptionsPlugin which hands the responsibilities of minifying to the loader, it would be perfect timing to add a minify feature into the loader.
Thoughts?
Cross reference post to both ts loaders.
s-panferov/awesome-typescript-loader#153
The text was updated successfully, but these errors were encountered: