-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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 for Rails 6+ ? #5037
Comments
I don't think this is a devise issue. I managed to get a Try checking out a clean branch, pinning rails to |
Yes 5.2.2 works fine. |
Hi @MatthiasRMS, thanks for the issue report. Yes, it is planned. See #5009, but we don't have a date to release it yet. Probably when we have the final Rails 6 release. It seems that responders already supports Rails 6, probably you are getting this error because it's not the final release of Rails. https://github.com/plataformatec/responders/blob/master/responders.gemspec#L21 |
I was migrating my project from rails 4.X to 6-beta2 and that was my roadblock. Went currently with Rails 5.2.2 but hopefully could hop on the 6 train soon when Devise supports it. |
Devise .travis.yml setting for rails 6 is
so it should be OK with responders but the test doesn't pass |
@saiqulhaq That's because Rails 6 beta 1 introduced a change in redirects that broke some of our specs. I started updating them but it seems like they have rolled that change back. |
Any idea when devise will support Rails 6? devise.gemspec strictly depends on railties < 6.0 |
Any estimate of when this will be released? |
@thenano I'll try to do it today. |
I've released version |
awesome, thanks so much @tegon ! |
Hi,
I'm trying to update my app to Rails 6 but I get this error:
Is it planned to update it for Rails 6 ?
I'd be happy to contribute to the upgrade and submit a PR if you already know what there is to do to maintain the compatibility.
Matthias
The text was updated successfully, but these errors were encountered: