-
Notifications
You must be signed in to change notification settings - Fork 140
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
Dependency on older Sinatra version #81
base: master
Are you sure you want to change the base?
Conversation
I ran in to this as well. You can also just loosen the requirement from ~> 1.0.0 to ~> 1.0. |
+1 This issue makes it really annoying to use taps with any rails app that is mounting Sinatra applications. |
Although looking more closely at this request. It doesn't seem prudent to totally remove the version requirements for all the gem dependencies. I like @mwynholds suggestion: just loosen up the version requirement for sinatra. |
+1 This really needs an update. |
+1 please get this in. Cannot use Taps in apps using Resque (app has both resque and db) (because Resque uses newer sinatra) |
+1 please! |
+1 please! Trying to use Magickly gem which requires newer Sinatra |
yes, please fix |
+1 |
4 similar comments
+1 |
+1 |
+1 |
+1 |
This needs to be addressed. Using Taps but I get an error because of a Sinatra bug using Tilt, which was fixed in later versions. But Taps only allows Sinatra 1.0.0. |
I just dropped the version information. Works fine for me.