1789033 - Running ./mach lint on a YAML file doesn't catch trailing whitespace

Status: RESOLVED FIXED

(Developer Infrastructure :: Lint and Formatting, enhancement)

Description

[Stephanie Cunnane [:scunnane]]

STR

1. Open a YAML file and add trailing whitespace to a line in the file
2. Run ./mach lint path/to/yaml/file

Expected results
The linter should catch the trailing whitespace.

Actual results
The linter doesn't catch the trailing whitespace.

While YAML with trailing whitespace is technically still valid YAML, this is a bit sloppy. We don't want to allow trailing whitespace to creep into the codebase.

Comment 1

[Mark Banner (:standard8)]

Attachment: Bug 1789033 - Fix whitespace issues for yaml and yml files. r?#linter-reviewers!

Comment 2

[Mark Banner (:standard8)]

Attachment: Bug 1789033 - Enable file-whitespace and file-perms on yml and yaml files. r?#linter-reviewers!

Depends on D194918

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/1e3c5f46ea60
Fix whitespace issues for yaml and yml files. r=linter-reviewers,sylvestre
https://hg.mozilla.org/integration/autoland/rev/de47344d8f97
Enable file-whitespace and file-perms on yml and yaml files. r=linter-reviewers,sylvestre

Comment 4

[Norisz Fay [:noriszfay]]

https://hg.mozilla.org/mozilla-central/rev/1e3c5f46ea60
https://hg.mozilla.org/mozilla-central/rev/de47344d8f97