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
Fixes #13074 - sprockets-rails should be in Gemfile #3047
Conversation
@@ -4,7 +4,6 @@ | |||
gem 'autoprefixer-rails', '~> 5.2' | |||
gem 'sprockets', '~> 3' |
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.
This is also a runtime dep, should it be moved to Gemfile also?
(See your PR description!)
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.
Thought I moved both, thanks!
@@ -39,6 +39,8 @@ end | |||
gem 'activerecord-session_store', '~> 0.1.1' | |||
gem 'rails-observers', '~> 0.1' | |||
gem 'protected_attributes', '~> 1.1.1' | |||
gem 'sprockets', '~> 3' | |||
gem 'sprockets-rails', '~> 2.3.3', '< 3' |
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.
Looks like a typo from aa50ab1, but this should be >=
and not ~>
else it'll never get beyond 2.3.x.
dependency out of the Gemfile and into assets.rb. sprockets-rails is a runtime dependency and so should stay in Gemfile. #12977 was a runtime error in production. The sprockets gem is also a runtime dependency, so should be in Gemfile rather than assets.rb.
Updated to use |
dependency out of the Gemfile and into assets.rb. sprockets-rails is a
runtime dependency and so should stay in Gemfile. #12977 was a runtime
error in production.
The sprockets gem is also a runtime dependency, so should be in Gemfile
rather than assets.rb.