-
-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
Added fix for multi composite key to V5 branch #12800
Conversation
It looks like |
Hello @cskiwi! I see you are a first-time contributor, thank you for taking the time to help Sequelize! I hope to see more PRs from you in the future! Sorry to take long to reply. I shall be replying faster now. Some tests are failing, can you fix them? |
hi @papb I would really love to run the tests, but I get following error:
Any suggestions? |
Pull Request check-list
Please make sure to review and check all of these items:
npm run test
ornpm run test-DIALECT
pass with this change (including linting)?Description of change
Added the fix from #11984 to the v5 branch.
This is needed for people using 'sequelize-typescript' because that doesn't work on v6 yet