Merge branch 'winh-scope-haml-linter' into 'master'

Restrict InlineJavaScript for haml_lint to dev and test environment

Closes omnibus-gitlab#2658

See merge request !13364

1 files changed, 11 insertions, 9 deletions

diff --git a/lib/haml_lint/inline_javascript.rb b/lib/haml_lint/inline_javascript.rb
index f3ddcbb9c95..05668c69006 100644
--- a/lib/haml_lint/inline_javascript.rb
+++ b/lib/haml_lint/inline_javascript.rb
@@ -1,14 +1,16 @@
-require 'haml_lint/haml_visitor'
-require 'haml_lint/linter'
-require 'haml_lint/linter_registry'
+unless Rails.env.production?
+  require 'haml_lint/haml_visitor'
+  require 'haml_lint/linter'
+  require 'haml_lint/linter_registry'
 
-module HamlLint
-  class Linter::InlineJavaScript < Linter
-    include LinterRegistry
+  module HamlLint
+    class Linter::InlineJavaScript < Linter
+      include LinterRegistry
 
-    def visit_filter(node)
-      return unless node.filter_type == 'javascript'
-      record_lint(node, 'Inline JavaScript is discouraged (https://docs.gitlab.com/ee/development/gotchas.html#do-not-use-inline-javascript-in-views)')
+      def visit_filter(node)
+        return unless node.filter_type == 'javascript'
+        record_lint(node, 'Inline JavaScript is discouraged (https://docs.gitlab.com/ee/development/gotchas.html#do-not-use-inline-javascript-in-views)')
+      end
     end
   end
 end