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

Sidekiq 4.0 #1

Closed
jnylen opened this issue Nov 14, 2015 · 10 comments
Closed

Sidekiq 4.0 #1

jnylen opened this issue Nov 14, 2015 · 10 comments
Assignees

Comments

@jnylen
Copy link

jnylen commented Nov 14, 2015

Hi,
Amazing gem you got here. Is it possible for you to look into moving it over to allow Sidekiq 4.0?
You need to drop the requirement of celluloid though.

@ixti
Copy link
Owner

ixti commented Nov 16, 2015

Well, sidekiq-throttled does not depend on celluloid directly.
And yes, I'm thinking of making it sidekiq 4.0 friendly. In fact,
we will support both 3.x and 4.x sidekiq branches.

Stay tuned! ;))

@ixti ixti self-assigned this Nov 16, 2015
@badlamer
Copy link

+1

@jnylen
Copy link
Author

jnylen commented Dec 13, 2015

Any ETA on this @ixti ?

@thatdom
Copy link

thatdom commented Dec 28, 2015

Thanks for great gem, any news on Sidekiq 4 support?

@ixti ixti added this to the 0.2.0 milestone Dec 30, 2015
@VishalRam
Copy link

+1

@jcran
Copy link

jcran commented Jan 13, 2016

+1

@ChaoticBoredom
Copy link

This looks awesome, appears to combine the utility of two different gems we're using. Supporting Sidekiq 4.0 would definitely be the topper.

@divoxx
Copy link

divoxx commented Jan 26, 2016

+1

@ixti ixti modified the milestones: 0.2.0, 0.3.0 Feb 29, 2016
@jnylen
Copy link
Author

jnylen commented Apr 22, 2016

Any ETA on this @ixti?

@ixti
Copy link
Owner

ixti commented Apr 25, 2016

Released with 0.3.0.pre. Please report issues if any. Will release 0.3.0 after a week.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

8 participants