-
Notifications
You must be signed in to change notification settings - Fork 15
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
Plugin doesn't work with angular 5.0.0-rc.0 #32
Comments
Propaply because of this: angular/angular#18966
|
The same problem also with 5.0.0-rc.1 |
Angular 5 will introduce lots of changes... I'll try to explore and see what we can add/improve |
This affects both awesome-typescript-loader and fork-ts-checker-webpack-plugin for Angular AOT builds, so, currently, in order to use Angular 5.0.0+, the only way forward is ngtools/webpack
|
Angular 5.0.0 final is out :) |
Bad news, they have changed the whole process, it's completely different. They have watch mode built in the compiler now which changed everything... I'll have to assess it and see. |
You can use @gregbown workaround for now. It means that:
|
OK, I'm trying to get to the best solution here and the situation is tricky and complex. I'll need you input here Some background:
Nowdays, most of the features are in the
From a process perspective,
This process has 2 major downsides:
This plays nicely with the new AOT watch mode feature in the To port Optional solutions:Here are the options, as I see them:
Now, i'd like to hear from you:
cc |
Well for now I will use it like this: https://ibb.co/m7eYfw |
@MADzO is it really possible? |
@Ks89 Yes it is... not ideal solution but working :) |
@MADzO probably it works. But it's terrible solution. Better stay several more days on Angular 4 if it is not super critical. |
Well, the solution is not optimal but not so bad. Reason is, the compilation is done by angular/compiler and the angular/compiler-cli is just the engine that runs it. So yes, there are thing happening there which are important like summery files, factory etc ... but it should be fine as you use something farily recent. For now it should probably do the trick until I push the fix |
@shlomiassaf Thank for you interest in this project and to try to update it to support Angular 5. It's really appreciated. |
@shlomiassaf - Thanks for sharing details. Also, Can you please share the plan to push these fixes. Do let me know if I can help you on this!! |
Hi guys, Thank you all for the support! I have refactored It will also allow easy migration to/from Please see the readme for more details. Use Thanks! |
@shlomiassaf - Thanks, Buddy!! Let me try it and get back to you with the result!! |
I have some question?
I never used ngtools.
|
Pushing this to angular-starter will take some time because it requires rearranging the plugins and loaders to be efficient. Running it however should be an issue. The Follow it from there to the base webpack config to get a clue on a valid setup |
@shlomiassaf add |
@shlomiassaf first of all, thank you for the update of angular-starter by angularclass. ngc-webpack supports multiple entry point (for instance 2 spa applications (main and admin))? |
@Ks89 So, if If it works there but not with Let me know if this is the case. |
@shlomiassaf no way for custom ts loaders here? tried to update medium size project (~70-80k loc) to ng5 from ng4 and got x2 prod build time (x3 with all build optimizations from angular team). ng4 build uses |
Seems angular team changed @angular/compiler-cli API since 5.0.0-rc.0.
The text was updated successfully, but these errors were encountered: