-
Notifications
You must be signed in to change notification settings - Fork 286
Drop sequelize version 3 support for future releases #157
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
Comments
I have the same opinion here!
+1
Martín
On Sat, Oct 07, 2017 at 12:01 Robin Buschmann < mailto:Robin Buschmann <[email protected]> > wrote:
a, pre, code, a:link, body { word-wrap: break-word !important; }
Supporting v3 and v4 at the same time makes the code base unnecessarily complex and improvements much more difficult. Since sequelize v4 is available for some month now, I think it makes sense to drop support for version 3.
https://github.com/bilby91 https://github.com/snewell92 https://github.com/brucehem
What do you think about it?
—
You are receiving this because you were mentioned.
Reply to this email directly,
#157
, or
https://github.com/notifications/unsubscribe-auth/ACGV988UUznPeEmAbMbEcqicJTgNYAc8ks5sp5IugaJpZM4PxYHC
.
|
@RobinBuschmann to be honest, I think it's best to drop support for v3. |
+1 |
I agree. And if someone needs v3 support (lock in on production or something), it would be easy to either 1, pin Let's do eeeeet! |
Sequelize version 3 support will be dropped with |
Dropped on |
Supporting v3 and v4 at the same time makes the code base unnecessarily complex and improvements much more difficult. Since sequelize v4 is available for some month now, I think it makes sense to drop support for version 3.
@bilby91 @snewell92 @BruceHem What do you think about it?
The text was updated successfully, but these errors were encountered: