-
Notifications
You must be signed in to change notification settings - Fork 11
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
chore: upgrade ng-mocks
to 14.12.1
#34
chore: upgrade ng-mocks
to 14.12.1
#34
Conversation
package.json
Outdated
@@ -21,7 +21,7 @@ | |||
"@angular/router": "17.0.2", | |||
"@testing-library/angular": "14.4.0", | |||
"@testing-library/user-event": "14.5.1", | |||
"ng-mocks": "14.11.0", | |||
"ng-mocks": "14.12.1", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh this brings to my attention that we need to cleanup the dependencies 😅
@testing-library packages needs to move to devDependencies, ng-mocks
isn't used and can thus be removed.
Do you want to update the PR with this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I can do that after work. It is 11:17 AM in my timezone so you will probably be sleeping by then. Otherwise I wouldn't mind if you want to do that now outside of the PR and close this.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There's no rush with it.
Please take your time 🙂
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I moved two @testing-library
packages to devDependencies, removed ng-mocks
, and updated the package-lock.json
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @michael-small !
🎉 This PR is included in version 1.11.3 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Errors occurred when trying to run ng-mocks 14.11.0. It appears that 14.12.0 was released to support Angular v17, and 14.12.1 is the latest version.
I have not done much else other than experiment with the example application. But now it seems to work.