Skip to content

Enable line limit linter

Pedro Pombeiro (OOO from July 16th till Aug 7th) requested to merge 25884-lll-linter into master

What does this MR do?

Enables the lll linter with a line limit of 120 characters.

Why was this MR needed?

To ensure that future contributions don't exceed the agreed limit.

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

I've fixed most of the reports, and added nolint:lll to those which don't warrant fixing (mostly JSON comments in structs, for instance).

Does this MR meet the acceptance criteria?

  • Documentation created/updated
  • Added tests for this feature/bug
  • In case of conflicts with master - branch was rebased

What are the relevant issue numbers?

Closes #25884 (closed)

Edited by Pedro Pombeiro (OOO from July 16th till Aug 7th)

Merge request reports