-
Notifications
You must be signed in to change notification settings - Fork 93
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
Bump angular versions to 14 #264
base: master
Are you sure you want to change the base?
Conversation
Refactor the project structure to comply recommendations
Thank you for all of your fantastic work porting to version 14! Overall this looks good although there a few minor updates that are required. I'll pull this down today and test it. |
@Stabzs Have you been able to look at this? |
Would be great if this would be released. We have a few projects > Angular 11 with "broken" dependencies. |
@dmkachko are you able to review and complete the rest of the feedback? |
This reverts commit 32e0f58.
@Stabzs Sorry accidentally pushed the next update to the same branch. |
@dmkachko no problem! If you make a separate branch/pr, I can make sure both get pushed out this week. Are you able to speak to this comment/question? "Where did the css transpilation move to and how is it exposed in the built package?" Thanks! |
@Stabzs I am sorry, just in case I am missing something, for some reason I don't see any conversations or change requests attached to the PR, and don't have anything in the mail either. Where do you see this question? |
EDIT: Found it...the request for changes didn't submit...my fault. Sorry about that! @dmkachko odd, not sure why that is. |
…n build scripts and deps
@Stabzs The feedback is implemented, pls let me know if you see anything else. |
Refactor the project structure to comply recommendations