-
-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
Changed variable name to string in getters/setters documentation #12811
Conversation
Shouldn't be username and password, but 'username' and 'password'
Codecov Report
@@ Coverage Diff @@
## master #12811 +/- ##
=======================================
Coverage 96.32% 96.32%
=======================================
Files 95 95
Lines 9261 9261
Branches 90 90
=======================================
Hits 8921 8921
Misses 323 323
Partials 17 17 Continue to review full report at Codecov.
|
There are similar changes in #12655. |
Hello! 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! Recently, we decided to undo some large work that was done on the |
🎉 This issue has been resolved in version 6.5.1 🎉 The release is available on: Your semantic-release bot 📦🚀 |
…12811 sequelize#12655) Co-authored-by: sliterok <korobatov2011@yandex.ru> Co-authored-by: William Gurzoni <william_luizat@hotmail.com>
…12811 sequelize#12655) Co-authored-by: sliterok <korobatov2011@yandex.ru> Co-authored-by: William Gurzoni <william_luizat@hotmail.com>
Shouldn't be username and password as variables, but 'username' and 'password' as strings