-
Notifications
You must be signed in to change notification settings - Fork 102
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
Angular #8
Comments
I've worked with Angular a fair amount. It's best used with |
Hey, awesome! That would be great! Although, I'd expect that to likely be an involved PR, just a heads up 😜 Does Angular now require TS, or is it just part of standard convention? Either way, I have a TS plugin mostly complete. If Angular does require TS, we'd build off/alongside that plugin as opposed to installing all of TS & co in the Angular preset a second time. |
This is an awesome project, and I'm definitely excited to see it grow to include an Angular preset :) AFAIK, you can't bootstrap a new Angular CLI project without TS. Some additional notes (in case it helps):
|
Awesome, thank you @housseindjirdeh ~! All sounds good, thanks. So then that definitely places #5 as a pre-req 👍 Those other behaviors should be working out of the box. If Angular works like 99.9% of everything else, it's just a matter of emitting the particular content type(s) into the Webpack sphere & allowing the delegated-n-designated loaders handle the rest. There's some good visibility here so far, so I'll definitely come back with questions since I haven't meddled w/ Angular since early 2.x 👻 That is, if @MarkRabey doesn't beat me to it 🙌 |
Sorry, I've hit a wall, so to speak, in terms of time. While I'd love to help out with this, it's probably best someone else take care of it. I've barely had time to think about it the past few days. |
Not to worry @MarkRabey! However much or little (or none) you're able to contribute is super appreciated. 👍 |
Add new
@pwa/preset-angular
package.I'm super out of touch with Angular, so this will need some research.
The text was updated successfully, but these errors were encountered: