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

Update to Rails 5.2.0 #912

Closed

Conversation

composerinteralia
Copy link
Contributor

@composerinteralia composerinteralia commented May 2, 2018

Rails 5.2.0 will give us ActiveStorage and all sorts of other goodies:
http://edgeguides.rubyonrails.org/5_2_release_notes.html

rails/rails#30016 introduced Rails::AppBuilder#ruby_version to create
a .ruby_version file. To avoid getting stuck in an overwrite prompt
when we try to create our own .ruby_version file based on
Suspenders::RUBY_VERSION, I overrode #ruby_version.

Rails 5.2.0 adds bootsnap to the Gemfile and to config/boot.rb. I added it
to our Gemfile, but we could also skip_bootsnap by default if we don't
want to use it.

Rails 5.2.0 will give us ActiveStorage and all sorts of other goodies:
http://edgeguides.rubyonrails.org/5_2_release_notes.html

rails/rails#30016 introduced `Rails::AppBuilder#ruby_version` to create
a .ruby_version file. To avoid getting stuck in an overwrite prompt
when we try to create our own .ruby_version file based on
`Suspenders::RUBY_VERSION`, I overrode `ruby_version`.

Rails 5.2.0 adds bootsnap to the Gemfile and to config/boot.rb. I added it
to our Gemfile, but we could also `skip_bootsnap` by default if we don't
want to use it.
@toobulkeh
Copy link
Contributor

Looks like you're missing the addition of a few lines for bootsnap to work:

https://github.com/Shopify/bootsnap#usage

@composerinteralia
Copy link
Contributor Author

@toobulkeh Rails 5.2 adds require 'bootsnap/setup' to config/boot.rb by default. Is there something else I am missing?

@mike-burns
Copy link
Contributor

Merged as d3c2e89, thank you!

@mike-burns mike-burns closed this May 25, 2018
@composerinteralia composerinteralia deleted the rails-5.2.0 branch May 25, 2018 18:39
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.

3 participants