Closed Bug 1789033 Opened 2 years ago Closed 3 months ago

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

Categories

(Developer Infrastructure :: Lint and Formatting, enhancement)

enhancement

Tracking

(firefox122 fixed)

RESOLVED FIXED
Tracking Status
firefox122 --- fixed

People

(Reporter: scunnane, Assigned: standard8)

References

Details

Attachments

(2 files)

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.

Assignee: nobody → standard8
Status: NEW → ASSIGNED
See Also: → 1867131
Pushed by mbanner@mozilla.com:
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
Status: ASSIGNED → RESOLVED
Closed: 3 months ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.