angular / angular2-seed

MIT License
1.01k stars 640 forks source link

feat: serve and bundle using webpack2. #132

Closed mprobst closed 7 years ago

mprobst commented 7 years ago

Including using the Angular AoT compiler ngc.

This does not hook up ngc with webpack yet, so probably should not be merged yet.

ogix commented 7 years ago

I wonder why did you leave only one webpack entry? For tree shaking to work? And I see polyfills are not loaded. Aren't they necessary?

mprobst commented 7 years ago

@ogix if you have tree shaking working off a root, you don't really need multiple binaries anymore. It's nicer to start off from the one main.ts file and just include whatever it needs.

@gdi2290 as I said above, this PR had some issues and should probably not be merged yet, in particular no non-AoT development mode. At this point it might be easier to use one of Rob's webpack starters. I'd suggest rather rolling back.