You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Originally posted by davidlj95 May 24, 2024
Currently to create an example app you call the script with the Angular version you want the example app created with. Noice, but that argument is linked to a hardcoded list of example apps.
This could be dynamic. The only thing impeding that rn is the standalone / non standalone argument. Which is what example apps hardcoded list is defined. If adding that as a CLI argument too:
No need to maintain a hardcoded list of example app configurations. The list of Angular CLI versions in the JSON file would be enough. Hence supporting a new Angular version is easier: a thing less to change
Standalone / non standalone can be created as wish. So we could create v16 with standalone an non-standalone for instance. Currently it's non standalone only as it's that version's CLI default.
The text was updated successfully, but these errors were encountered:
Eventually opted to go for just creating the Angular app as the Angular CLI does. Knowing that for v17 onwards will be standalone and otherwise module-based. This way no need to specify an extra param when calling the script + the verification if that Angular CLI version can create standalone apps. Indeed --standalone for new apps was introduced in v16. It's actually only Angular CLI v16 the major version that allows can create standalone apps but it's disabled by default
Discussed in #607
Originally posted by davidlj95 May 24, 2024
Currently to create an example app you call the script with the Angular version you want the example app created with. Noice, but that argument is linked to a hardcoded list of example apps.
This could be dynamic. The only thing impeding that rn is the standalone / non standalone argument. Which is what example apps hardcoded list is defined. If adding that as a CLI argument too:
The text was updated successfully, but these errors were encountered: