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 support angular 16 #299

Merged
merged 1 commit into from
Jul 3, 2023

Conversation

erikmom
Copy link
Contributor

@erikmom erikmom commented Jul 3, 2023

No description provided.

@maxime1992
Copy link
Contributor

LGTM, thanks for the PR.

Please rebase the commits in 1 commit and mark that commit as a breaking change (see previous commits on major version bumps for ngx sub form for examples).

This is for semantic versioning as we've got automatic releases and it wouldn't bump to v9 of the lib otherwise

BREAKING CHANGE: Angular 16 required

This closes cloudnc#298

Also wanted to to upgrade typescript to a 5.x.x version, but the Cypress typescript compilation failed, because of cypress-io/cypress#26203
@erikmom
Copy link
Contributor Author

erikmom commented Jul 3, 2023

@maxime1992 I've squashed the commits and updated the commit message.

Let me know if I need to do more!

@maxime1992 maxime1992 merged commit 936e017 into cloudnc:master Jul 3, 2023
@maxime1992
Copy link
Contributor

LGTM thanks :)

@maxime1992
Copy link
Contributor

@erikmom after the upgrade master started to fail. I dug into this for about an hour and still no success.

Can you please have a look on your side? If it's a quick fix awesome, if it's not I'll have to revert the changes until we can figure it out

@erikmom
Copy link
Contributor Author

erikmom commented Jul 5, 2023

Hey @maxime1992,

I saw in Github actions that the linting failed and I could reproduce it locally as well.

I've recreated the pull request with two fixes:

  • Used a different version of yarn (now using v1.22.19). I think the package lock file now looks more similar to what it was on version 8 of ngx-subform.
  • Removed linting configuration to extend from ng-cli-compat. As can be read here, that configuration is no longer supported.

@github-actions
Copy link

github-actions bot commented Jul 6, 2023

🎉 This PR is included in version 9.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants