Skip to content

Enable HAML-LINT linters

kushalpandya requested to merge 22072-enable-haml-lints into master

What does this MR do?

This MR enables new HAML Linters as mentioned in #22072 (closed) (particularly in this comment)

  • AltText
  • ClassAttributeWithStaticValue
  • FinalNewline
  • HtmlAttributes
  • ImplicitDiv
  • SpaceBeforeScript
  • TrailingWhitespace
  • SpaceInsideHashAttributes

Are there points in the code the reviewer needs to double check?

Pretty much every change needs to be reviewed as we want to be sure that an attempt to fix a Lint rule doesn't cause any unintended side-effect (which otherwise doesn't happen)

Why was this MR needed?

See #22072 (closed) and discussion thread in same issue.

Does this MR meet the acceptance criteria?

What are the relevant issue numbers?

Closes #22072 (closed)

Merge request reports