lykmapipo / kue-scheduler

A job scheduler utility for kue, backed by redis and built for node.js
246 stars 47 forks source link

Kue update #23

Closed KeKs0r closed 8 years ago

KeKs0r commented 8 years ago

Here my PR to update Kue to 10.3

I have tried the new Kue version on staging, but not yet on Production. Just want to open it to put it on the roadmap and start evaluating it.

lykmapipo commented 8 years ago

@KeKs0r

I will work on it this weekend. Hope you have checked 0.9 -> 0.10 upgrade.

Thanks for PR.

KeKs0r commented 8 years ago

@lykmapipo I did take care of this for my environment. But I think if we tackle this we should make it a new kue-scheduler major version and add the same release notes.

lykmapipo commented 8 years ago

@KeKs0r

Sure. That was actually the purpose of my question. We will have to update major version and add release notes.

:+1:

KeKs0r commented 8 years ago

Hey @lykmapipo can we move forward with this? The kue 9->10 upgrade only states that you should start with a clean redis instance. So the same applies for upgrading kue-scheduler.

elyas-bhy commented 8 years ago

+1, looking forward to this PR.

lykmapipo commented 8 years ago

@KeKs0r & @elyas-bhy

I will work on this. It will be out after fix the merge

KeKs0r commented 8 years ago

I also did the update for the devdependencies in order to see the actual test output. Seems that we still have some issues with it.

KeKs0r commented 8 years ago

@lykmapipo : ping

KeKs0r commented 8 years ago

@lykmapipo : I guess this can be closed, since you updated kue to 10.5