Skip to content
This repository has been archived by the owner on Mar 7, 2018. It is now read-only.

Commit

Permalink
Lock Dashing on Rufus Scheduler 2.0.24. There are some small issues f…
Browse files Browse the repository at this point in the history
…rom using the rewritten RS (3.0). So for now let's use this version until Dashing 2.0 comes out.
  • Loading branch information
pushmatrix committed May 30, 2014
1 parent 391a270 commit ea3730f
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion dashing.gemspec
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ Gem::Specification.new do |s|
s.add_dependency('sinatra', '~> 1.4.4')
s.add_dependency('sinatra-contrib', '~> 1.4.2')
s.add_dependency('thin', '~> 1.6.1')
s.add_dependency('rufus-scheduler', '~> 3.0.3')
s.add_dependency('rufus-scheduler', '~> 2.0.24')
s.add_dependency('thor', '~> 0.18.1')
s.add_dependency('sprockets', '~> 2.10.1')
s.add_dependency('rack', '~> 1.5.2')
Expand Down

7 comments on commit ea3730f

@airhorns
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@pushmatrix do you remember what issues these were? Can we patch upstream? I have a lot more confidence in Rufus 3

@pushmatrix
Copy link
Member Author

@pushmatrix pushmatrix commented on ea3730f Jun 19, 2014 via email

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@gottfrois
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Had the same issue with dashing-rails. Updated to 3.0 and had to do first_in: 1.second.since

@JonMcPherson
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This shouldn't be an issue anymore after the next release. I submitted a PR and it got merged.
jmettraux/rufus-scheduler#179

@jmettraux
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@JonMcPherson
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the fast response and release John!

@jmettraux
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You're welcome!

Please sign in to comment.