2003775 - Upgrade ktlint to version 1.8.0

Status: NEW

(Firefox for Android :: Tooling, task)

Description

[Ryan VanderMeulen [:RyanVM]]

Release notes: https://github.com/pinterest/ktlint/releases/tag/1.8.0

A couple of the newly-promoted checks are pretty noisy for our codebase, so I'm punting on enabling them for now. But I did take this as an opportunity to get our configs more closely aligned across the various projects with the hopes of eventually getting everything into a single shared one.

Comment 1

[Ryan VanderMeulen [:RyanVM]]

Green on Try: https://treeherder.mozilla.org/jobs?repo=try&revision=64e19d49f64445132deb46481a8374e58cfbd9e7

Comment 2

[Ryan VanderMeulen [:RyanVM]]

Attachment: Bug 2003775 - Update ktlint to version 1.8.0. r=#android-reviewers

Comment 3

[Ryan VanderMeulen [:RyanVM]]

Attachment: Bug 2003775 - Fix some new errors. r=#android-reviewers

Comment 4

[Ryan VanderMeulen [:RyanVM]]

Attachment: Bug 2003775 - Fix filename errors and remove ktlint baselines. r=#android-reviewers

Comment 5

[Ryan VanderMeulen [:RyanVM]]

Attachment: Bug 2003775 - Use more consistent ktlint configs. r=#android-reviewers

Eventually we should unify them under a single shared config, but this at least
gets us using consistent rules across the different configs as a starting point.

Comment 6

[Ryan VanderMeulen [:RyanVM]]

Attachment: Bug 2003775 - Fix argument-list-wrapping errors. r=#android-reviewers

Comment 7

[Ryan VanderMeulen [:RyanVM]]

Attachment: Bug 2003775 - Fix parameter-list-wrapping errors. r=#android-reviewers

Comment 8

[Ryan VanderMeulen [:RyanVM]]

Attachment: Bug 2003775 - Fix no-blank-line-in-list errors. r=#android-reviewers