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(types): add typescript 4.8 compatibility #14990
Conversation
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.
Can you write types tests for this?
Thanks for the PR! Are you willing to make a PR towards the main branch as well? |
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.
Other than that, if tests pass (and they do), lgtm!
I think the type test suite here should already cover everything that has been impacted https://github.com/sequelize/sequelize/tree/main/test/types
6660a18
to
aa3bfae
Compare
Failing tests are flaky tests, we can merge |
🎉 This PR is included in version 6.23.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Thank you for the initiatives! |
Pull Request Checklist
Please make sure to review and check all of these items:
yarn test
oryarn test-DIALECT
pass with this change (including linting)?Description Of Change
Sequelize typings are affected in a few places by a TypeScript 4.8 breaking change (https://devblogs.microsoft.com/typescript/announcing-typescript-4-8/#unconstrained-generics-no-longer-assignable-to).
This PR addresses #14934.