Merge pull request #9286 from kbrock/drop_sass_rails

Drop sass rails

Author: Fryguy

config/initializers/assets.rb

@@ -1,5 +1,5 @@
 Rails.application.config.assets.paths << ManageIQ::UI::Classic::Engine.root.join('node_modules')
 
 Rails.application.config.assets.precompile << proc do |filename, path|
-  path =~ %r{app/assets} && !%w(.js .css).include?(File.extname(filename))
+  path =~ %r{app/assets} && !%w(.js .css .sass).include?(File.extname(filename))
 end

lib/manageiq/ui/classic/engine.rb

@@ -1,18 +1,6 @@
 require 'rails/engine'
 require 'sprockets/railtie'
 
-# Since we serve our assets directly through apache on an appliance after they
-# are pre-compiled, there is no need to have sass/coffeescript loaded in the
-# application, so we can save a bit of resources by not loading these two.
-#
-# That said, we still need to load both of these when pre-compiling the assets
-# in production mode, so if Rake is defined, load things like we used to.
-#
-# For this to work properly, it is dependent on patternfly/patternfly-sass#150
-if ENV["RAILS_ENV"] != "production" || defined?(Rake)
-  require 'sass-rails'
-end
-
 require 'high_voltage'
 require 'webpacker'
 

manageiq-ui-classic.gemspec

@@ -27,7 +27,7 @@ Gem::Specification.new do |s|
   # TODO: fix manageiq-decorators so we can eager load after manageiq-ui-classic
   # s.add_dependency "manageiq-decorators"
   s.add_dependency "more_core_extensions", ">= 3.2", "< 5"
-  s.add_dependency "sass-rails"
+  s.add_dependency "sprockets-rails"
   s.add_dependency "uglifier", "~>4.2.0"
   s.add_dependency "webpacker", "~>2.0.0"