From 25283e71011490c674c54e64384957a692235e38 Mon Sep 17 00:00:00 2001 From: Connor Shea Date: Wed, 28 Feb 2018 15:58:12 -0700 Subject: Update inline_javascript.rb to lint uses of the javascript filter as well as script tags. --- lib/haml_lint/inline_javascript.rb | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) (limited to 'lib/haml_lint/inline_javascript.rb') diff --git a/lib/haml_lint/inline_javascript.rb b/lib/haml_lint/inline_javascript.rb index ff447513fd1..4f776330e80 100644 --- a/lib/haml_lint/inline_javascript.rb +++ b/lib/haml_lint/inline_javascript.rb @@ -6,7 +6,13 @@ unless Rails.env.production? 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)') + end + def visit_tag(node) return unless node.tag_name == 'script' -- cgit v1.2.1