Skip to content
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

feat: extend version support range to Angular 18 #2462

Merged
merged 1 commit into from
May 20, 2024

Conversation

leosvelperez
Copy link
Contributor

@leosvelperez leosvelperez commented May 20, 2024

Summary

Angular 18 should be released sometime this week (the Week of November 20th, 2024). This PR allows users to install jest-preset-angular without getting peerDependency errors, which cause some package managers to fail the installation process.

Test plan

We've tested jest-preset-angular + Angular 18 as part of the Angular 18 support for Nx, and everything works other than a peer dependency error.

Additionally, I created a separate PR with a new example test application using the latest RC version of Angular 18: #2463. The CI checks are all passing. I'll update that PR when the stable version of Angular 18 is released.

Does this PR introduce a breaking change?

  • Yes
  • No

Other information

Copy link
Collaborator

@ahnpnl ahnpnl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@ahnpnl ahnpnl merged commit 24e097d into thymikee:main May 20, 2024
9 checks passed
@leosvelperez leosvelperez deleted the feat/angular-18 branch May 21, 2024 07:00
@leosvelperez
Copy link
Contributor Author

Thank you! Do you think you could release the change? It would allow us at Nx to support the new version before the Angular release this week. Given that this PR doesn't change any functionality, the change could be released before the Angular version is released.

@ahnpnl
Copy link
Collaborator

ahnpnl commented May 21, 2024

just did 👍

@leosvelperez
Copy link
Contributor Author

Awesome! Thanks!

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.

2 participants