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
Migrate to Sprockets 4. #21132
Migrate to Sprockets 4. #21132
Conversation
@@ -0,0 +1,4 @@ | |||
|
|||
//= link_tree ./images | |||
//= link ./javascripts/application |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I remember we had problems because we didn't make explicit the extension right?
a0664f8
to
c74a9c4
Compare
@@ -20,7 +20,7 @@ gem 'turbolinks' | |||
gem 'arel', github: 'rails/arel', branch: 'master' | |||
gem 'mail', github: 'mikel/mail' | |||
|
|||
gem 'sprockets', '~> 3.0.0.rc.1' | |||
gem 'sprockets', github: 'rails/sprockets', branch: 'master' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we need to add sass-rails master here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Perhaps we could add it here also, it is already present in the Gemfiles of the generated applications and I don't fully understand what gems should be present in Gemfile for the rails gem itself, so I didn't include it because no test was failing because of this.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If we need on generate Gemfiles we need here too. The Railties tests use this Gemfile instead of the generated one.
754d76a
to
1926b1f
Compare
Depends on: rails/sprockets#85
cc: @rafaelfranca