diff options
author | Lin Jen-Shin <godfat@godfat.org> | 2018-10-24 15:29:03 +0800 |
---|---|---|
committer | Lin Jen-Shin <godfat@godfat.org> | 2018-10-24 15:31:00 +0800 |
commit | 3a9271f17914858879221a141989bb04076c0b0c (patch) | |
tree | 9d5b093b0b508850d1ffd10d578684c358e6b545 /lib/tasks | |
parent | 0cbefe85ed2823d2201fd9d30120ecf7535e88a0 (diff) | |
download | gitlab-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.rake | 11 |
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 |