-
Notifications
You must be signed in to change notification settings - Fork 2.5k
Release 2.0 stable #526
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
+1 On Fri, Jun 28, 2013 at 12:37 AM, Felix Geisendörfer
|
Hey Felix, a final version would be pretty nice to have - you could also spin up (some) betas prior. It would be really nice though to have 'prepared statements' working during the alpha/beta phase; in case the API has to change ... just my 2 cents :) PS: Is there already a branch you/or somebody started working on? Maybe I can help ... |
Well, the funding status of this is a bit unclear again, so I don't know when / if I'll have time to work on it. We can always release 3.0 if we have to break the API for prepared statements (I dont' think we'll have to, they'll be a simple addition). Anyway, I did get the very basics going over here: https://github.com/felixge/node-mysql/tree/prepare - if somebody wants to pick it up - go for it. It's mainly all the different types that need implementing. |
I think that it would be good if #505 was included. I will check more. After that, try to merge. There are no objections about 2.0 stable release. |
I also think there should be a stable 2.0 soon, but some issues should be fixed first. Maybe we should create a milestone? |
👍 |
I agree with @tellnes , a milestone would be nice. Maybe we could make a sprint and try to close bugs on Poll. |
Created a milestone and added a few issues to it. |
@tellnes <3 thanks. Everybody else: Should we add anything else to this list? |
@felixge and everybody else: Probably yes. I just added a few issues at the top of the issue tracker. |
Looks good for now :) |
please take a look at #539. There is a potential breaking change proposed which would be good to be included prior to cutting a stable release. |
Just found this discussion while searching for pool idle disconnect options ( #505 ). I don't think it fits 2.0 release (it adds new api) but I really want to see it in 2.1 |
Let's open a new issue when needed to discuss 2.1. 2.0 is done :) |
@dresende I'm planning to release more often if you don't mind, no need to have discussion for releases. On the other hand, let's have a rule of at least two 👍 on non-trivial pull request before merging |
Yes, no need for a discussion for every release. I was just wanted to close the issue. Any other issues should be addressed in new ones :) I agree with at least 2 👍 |
Hi guys,
do you guys have any objections against cutting a 2.0 stable release at some point? Do we have some APIs that we need decisions on?
It would be really nice if
npm install mysql
would start doing the right thing : ).cc @dresende @ifsnow @kai-koch
The text was updated successfully, but these errors were encountered: