From 2f85645d453ea67fac349325ff062735e758ddac Mon Sep 17 00:00:00 2001 From: Tim Zallmann Date: Fri, 14 Jul 2017 19:58:54 +0200 Subject: Brought in HAML Linter changes from Winnie --- .haml-lint.yml | 3 +++ lib/haml_lint/inline_javascript.rb | 14 ++++++++++++++ lib/tasks/haml-lint.rake | 1 + 3 files changed, 18 insertions(+) create mode 100644 lib/haml_lint/inline_javascript.rb diff --git a/.haml-lint.yml b/.haml-lint.yml index 528f99d08d2..09a11f4f7c1 100644 --- a/.haml-lint.yml +++ b/.haml-lint.yml @@ -38,6 +38,9 @@ linters: ImplicitDiv: enabled: true + InlineJavaScript: + enabled: true + LeadingCommentSpace: enabled: false diff --git a/lib/haml_lint/inline_javascript.rb b/lib/haml_lint/inline_javascript.rb new file mode 100644 index 00000000000..eef95cd95ef --- /dev/null +++ b/lib/haml_lint/inline_javascript.rb @@ -0,0 +1,14 @@ +require 'haml_lint/haml_visitor' +require 'haml_lint/linter' +require 'haml_lint/linter_registry' + +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. If needed, you can add this file to the list of exceptions in .haml-lint.yml.') + end + end +end diff --git a/lib/tasks/haml-lint.rake b/lib/tasks/haml-lint.rake index 609dfaa48e3..ad2d034b0b4 100644 --- a/lib/tasks/haml-lint.rake +++ b/lib/tasks/haml-lint.rake @@ -1,5 +1,6 @@ unless Rails.env.production? require 'haml_lint/rake_task' + require 'haml_lint/inline_javascript' HamlLint::RakeTask.new end -- cgit v1.2.1