-
Notifications
You must be signed in to change notification settings - Fork 3
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
Add correct ng annotate loader #4
Conversation
@@ -4,6 +4,7 @@ const HtmlPlugin = require('html-webpack-plugin') | |||
const NgAnnotatePlugin = require('ng-annotate-webpack-plugin') | |||
const OmitTildeWebpackPlugin = require('omit-tilde-webpack-plugin') | |||
const webpack = require('webpack') | |||
const combineLoaders = require('webpack-combine-loaders') |
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.
vraiment nécessaire ?
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.
mais oui je comprends l'intéret. je n'ai rien dit.
LGTM so far |
1738357
to
96ac674
Compare
|
||
The angular profile add the ng-annotate loader. | ||
FIXME(rboucher) | ||
`ng-annotate` **does not support arrow functions** for now. |
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.
donne juste un exemple de cas où ça ne fonctionne pas.
96ac674
to
3f12416
Compare
WIP
Todo : make private ng-annotate package until this PR gets merged