Add support for Angular 16#15925
Conversation
amakhrov
left a comment
There was a problem hiding this comment.
Thanks for your PR!
Looks good overall - just a few questions regarding dependencies versions
README.md
Outdated
There was a problem hiding this comment.
not really a blocker - but looks like the min version the angular generator supports is 9.x, not 2.x
There was a problem hiding this comment.
not sure how exactly it's used in the templates. but the min required version is 0.13.x, as per the migration guide:
https://update.angular.io/?l=3&v=15.0-16.0
Angular v16 supports Zone.js version 0.13.x or later.
There was a problem hiding this comment.
Indeed, I've fixed the dependencies specifications.
There was a problem hiding this comment.
passing note, not a blocker
no idea why this module is needed at all. the services are provided in root.
without specifying a custom config it's meaningless. if we need to provide a custom config, it can be done directly with an injection token.
using a module for this purpose doesn't match modern practices in Angular itself
There was a problem hiding this comment.
Removing the module might be breaking for existing projects; using the module itself is anyways optional I think. A modern project doesn't need to import the module at all.
There was a problem hiding this comment.
Maybe something to consider for a next major version
64843a3 to
c7b54c8
Compare
|
@macjohnny looks like it's ready for merging |
|
Any estimates when a version containing this change will be released? |
|
I'll try to release v7.0.0-beta this week |
The dependencies have been updated accordingly.
@TiFu (2017/07) @taxpon (2017/07) @sebastianhaas (2017/07) @kenisteward (2017/07) @Vrolijkx (2017/09) @macjohnny (2018/01) @topce (2018/10) @akehir (2019/07) @petejohansonxo (2019/11) @amakhrov (2020/02) @davidgamero (2022/03) @mkusaka (2022/04)
PR checklist
This is important, as CI jobs will verify all generator outputs of your HEAD commit as it would merge with master.
These must match the expectations made by your contribution.
You may regenerate an individual generator by passing the relevant config(s) as an argument to the script, for example
./bin/generate-samples.sh bin/configs/java*.For Windows users, please run the script in Git BASH.
master(6.3.0) (minor release - breaking changes with fallbacks),7.0.x(breaking changes without fallbacks)