Merge pull request #5546 from dmarcoux/fix_haml-lint_MultilineScript_…

…offense Fix haml-lint MultilineScript offense
openSUSE · Aug 3, 2018 · 348557b · 348557b
2 parents abb43ef + 8864662
commit 348557b
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 7 deletions.
diff --git a/src/api/.haml-lint_todo.yml b/src/api/.haml-lint_todo.yml
@@ -41,11 +41,6 @@ linters:
       - "app/views/webui/user/show.html.haml"
       - "app/views/webui/cloud/azure/configurations/show.html.haml"
 
-  # Offense count: 1
-  MultilineScript:
-    exclude:
-      - "app/views/webui/package/_rpmlint_log.html.haml"
-
   # Offense count: 6
   MultilinePipe:
     exclude:

diff --git a/src/api/app/views/webui/package/_rpmlint_log.html.haml b/src/api/app/views/webui/package/_rpmlint_log.html.haml
@@ -1,7 +1,7 @@
 - @log.lines.each do |line|
-  - if line =~ /\w+(?:\.\w+)+: W: /
+  - if line.match?(/\w+(?:\.\w+)+: W: /)
     = content_tag(:span, line.strip, style: 'color: olive;')
-  - elsif line =~ /\w+(?:\.\w+)+: E: /
+  - elsif line.match?(/\w+(?:\.\w+)+: E: /)
     = content_tag(:span, line.strip, style: 'color: red;')
   - else
     = line.strip