Skip to content

RuboCop: Enable previously disabled cop Layout/SpaceInLambdaLiteral

Adam Jensen requested to merge 369268-rubocop-reneable-Layout/SpaceInLambdaLiteral into master

What does this MR do and why?

Re-enables the Layout/SpaceInLambdaLiteral cop as requested in #369268 (closed) and put it in grace period.

It also fixes all offenses in HAML files.

Done via:

  1. Add Details: grace period to .rubocop_todo/layout/space_in_lambda_literal.yml
  2. Regenerate todos via rake rubocop:todo:generate[Layout/SpaceInLambdaLiteral]
  3. Fix offenses in HAML files
unset REVEAL_RUBOCOP_TODO

files=$(bundle exec haml-lint --parallel {,ee/}app/views | rg -o -r '$1' '^([^:]+):\d+' | uniq)

sed -i 's/-> /->/g' $files

bundle exec haml-lint --parallel {,ee/}app/views
2 files inspected, 0 lints detected

How to set up and validate locally

bundle exec rubocop --only Layout/SpaceInLambdaLiteral
bundle exec haml-link {,ee/}app/views

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Related to #369268 (closed)

Edited by Peter Leitzen

Merge request reports