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

Upgrade angular 13 ivy only #230

Merged
merged 17 commits into from
Nov 18, 2021
Merged

Conversation

maxime1992
Copy link
Contributor

No description provided.

BREAKING CHANGE:

By upgrading to Angular 13, ngx-sub-form will now stop publishing view engine compatible bundle.
If you're still running a pre Angular 13 version and want to use ngx-sub-form from here it should still be safe to do so thanks to ngcc.
If you're on Angular 13 already then when you install the dependencies by running `npm install` or `yarn` you shouldn't see `ngx-sub-form` in the list of libraries being compiled by ngcc anymore.
@maxime1992 maxime1992 force-pushed the upgrade-angular-13-ivy-only branch from 9b3b6d5 to 982d493 Compare November 18, 2021 16:24
@Waterstraal
Copy link
Contributor

❌ Linting failed

@maxime1992
Copy link
Contributor Author

x Linting failed

Yep, migrated to eslint but went wrong. There was loads of errors which I started to solve but it was way too long so I've turned off the new rules for now with a comment to restore them later. Hopefully it's good now!

@maxime1992 maxime1992 force-pushed the upgrade-angular-13-ivy-only branch from a31151f to 7a87fbc Compare November 18, 2021 21:21
@zakhenry zakhenry self-requested a review November 18, 2021 21:27
@zakhenry zakhenry merged commit bcf1d75 into feat-rewrite Nov 18, 2021
@zakhenry zakhenry deleted the upgrade-angular-13-ivy-only branch November 18, 2021 21:27
@github-actions
Copy link

🎉 This PR is included in version 6.0.0-feat-rewrite.17 🎉

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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants