Skip to content

FR-12555 - support-angular-16-backward-compatible #449

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

Merged
merged 1 commit into from
Aug 9, 2023

Conversation

yuvalotem1
Copy link
Contributor

@yuvalotem1 yuvalotem1 commented Aug 6, 2023

Support angular 16 backward compatible without signals

  1. enable ivy by removing "enablvy": false to support angular 16 build - angular 16 drop support to view engine so only ivy engine is enabled checkout angular CHANGELOG.md file-
    image

  2. Change angular version from ~12.0.5 to ^12.0.0 to remove error build of incompatible angular versions with ivy engine

  3. Avoid updating angular version to 16 because this will drop support for previous versions

How should we test it

  • tested on angular 12 + 14 + 16 during development
  • should be tested also for 13 and 15
  • also test full sanity for angular 16

Releasing

  • In my opinion this version should be a major 5 => 6 just like angular version 15 => 16 , this way it will make it very clear to our customers that this version is the one compatible with angular 16 when upgrading their angular version

after checking this pr move on to the signals support pr - #450

@yuvalotem1 yuvalotem1 changed the title FR-12555 - minimal-support-for-angular-16 FR-12555 - support-angular-16-with-backward-compatibility Aug 7, 2023
@yuvalotem1 yuvalotem1 changed the title FR-12555 - support-angular-16-with-backward-compatibility FR-12555 - support-angular-16-backward-compatibile Aug 7, 2023
@yuvalotem1 yuvalotem1 changed the title FR-12555 - support-angular-16-backward-compatibile FR-12555 - support-angular-16-backward-compatible Aug 7, 2023
@yuvalotem1 yuvalotem1 merged commit f86b504 into master Aug 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants