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

Update sequelize v5 #40

Merged
merged 2 commits into from
Apr 15, 2019
Merged

Update sequelize v5 #40

merged 2 commits into from
Apr 15, 2019

Conversation

joshuaalpuerto
Copy link
Owner

This PR should remove the warning about insecure dependency.

  • Upgraded sequelize@5.5.0 as recommended version.
  • Fix Breaking change to repositories.

More info about the Breaking change of migrating from v4 to v5 you can check out here

…o v5

BREAKING CHANGE: Underscore is removed from config, findById -> findByPk
@joshuaalpuerto joshuaalpuerto merged commit 951a8a9 into master Apr 15, 2019
@coveralls
Copy link

Coverage Status

Coverage remained the same at 90.713% when pulling 3fa8c91 on update-seqeulize into b9bb5e8 on master.

@joshuaalpuerto joshuaalpuerto deleted the update-seqeulize branch April 15, 2019 16:06
@joshuaalpuerto
Copy link
Owner Author

🎉 This PR is included in version 3.0.0 🎉

The release is available on GitHub release

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.

None yet

2 participants