-
Notifications
You must be signed in to change notification settings - Fork 63
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
Support Ruby 3.0 #114
Comments
A wild guess: https://eregon.me/blog/2021/02/13/correct-delegation-in-ruby-2-27-3.html This blog post can help fix this issue. |
The majority of those errors are related to the Mustermann extension for Sinatra That extension is only valid for Sinatra versions under 2.0 because from 2.0 onwards Mustermann is used by default so updating this code to work with Ruby 3 would mean giving Ruby 3.0 support for legacy Sinatra versions, is that really needed/desired? Sinatra 2.0 was released 4 years ago so IMHO we have 2 alternatives, either we get rid of this extension because it isn't needed anymore or we extract the extension to its own gem. WDYT? PD: I may be completely wrong in my analysis, a second look is more than welcome 😄 |
Getting rid of it sounds strong! Extraction seems like a good secondary option! |
😅 yeah maybe the second option is better, for it we would need to create a new repository in the organization because if we create the gem under this monorepo we will find the same issues (running the test suite with 3 will fail) right? |
Maybe move it to a separate gem and officially deprecate it? |
@epergo Quick work! I like this direction. |
Nice! I've forked it to the org: https://github.com/sinatra/mustermann-sinatra-extension |
cc #72 |
This Issue is a placeholder for the test failures seen in https://travis-ci.org/github/sinatra/mustermann/jobs/766897981
We recently added (#112) 3.0 to the build matrix, and allowed failures for it.
Now, we would like to fix the test failures!
All the 75 test failures, right here
The text was updated successfully, but these errors were encountered: