summaryrefslogtreecommitdiff
path: root/lib/tasks
diff options
context:
space:
mode:
authorLin Jen-Shin <godfat@godfat.org>2018-10-24 15:29:03 +0800
committerLin Jen-Shin <godfat@godfat.org>2018-10-24 15:31:00 +0800
commit3a9271f17914858879221a141989bb04076c0b0c (patch)
tree9d5b093b0b508850d1ffd10d578684c358e6b545 /lib/tasks
parent0cbefe85ed2823d2201fd9d30120ecf7535e88a0 (diff)
downloadgitlab-ce-3a9271f17914858879221a141989bb04076c0b0c.tar.gz
Convert parser warnings to stdout in haml_lintsh-bump-ruby-2.4.5
This was used in https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/16648 and this could happen from time to time, so let's just keep it.
Diffstat (limited to 'lib/tasks')
-rw-r--r--lib/tasks/haml-lint.rake11
1 files changed, 11 insertions, 0 deletions
diff --git a/lib/tasks/haml-lint.rake b/lib/tasks/haml-lint.rake
index ad2d034b0b4..786efd14b1a 100644
--- a/lib/tasks/haml-lint.rake
+++ b/lib/tasks/haml-lint.rake
@@ -2,5 +2,16 @@ unless Rails.env.production?
require 'haml_lint/rake_task'
require 'haml_lint/inline_javascript'
+ # Workaround for warnings from parser/current
+ # Keep it even if it no longer emits any warnings,
+ # because we'll still see warnings in console/server anyway,
+ # and we don't need to break static-analysis for this.
+ task :haml_lint do
+ require 'parser'
+ def Parser.warn(*args)
+ puts(*args) # static-analysis ignores stdout if status is 0
+ end
+ end
+
HamlLint::RakeTask.new
end