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

Rails 5, Alchemy 4, Devise 4 #58

Merged
merged 9 commits into from
Jul 17, 2017
Merged

Rails 5, Alchemy 4, Devise 4 #58

merged 9 commits into from
Jul 17, 2017

Conversation

tvdeyen
Copy link
Member

@tvdeyen tvdeyen commented Jul 13, 2017

Updates to Alchemy 4 and Devise 4 with support for Rails 5.

@tvdeyen
Copy link
Member Author

tvdeyen commented Jul 13, 2017

The vast majority of these changes are the Rails 5 updates of the dummy app.

@tvdeyen tvdeyen requested a review from mamhoff July 13, 2017 17:31
Copy link
Contributor

@mamhoff mamhoff left a comment

Choose a reason for hiding this comment

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

Go for it, I see no blockers here.

These versions support Rails 5
Some specs are depending on it.
We tackle deprecations later
Rails 5 controller tests does not work with rspec request stubs
anymore. Needed to stub ActionController::TestRequest instead.
In order to test deliver_later properly we use the test queue adapter.
- Only build ruby versions that are compatible with Rails 5
- Uses trusty distro
@tvdeyen tvdeyen merged commit a2e71b7 into master Jul 17, 2017
@tvdeyen tvdeyen deleted the alchemy-4 branch July 17, 2017 07:29
@tvdeyen tvdeyen added this to the 4.0 milestone Nov 6, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants