Closed
Bug 1789033
Opened 2 years ago
Closed 6 months ago
Running ./mach lint on a YAML file doesn't catch trailing whitespace
Categories
(Developer Infrastructure :: Lint and Formatting, enhancement)
Developer Infrastructure
Lint and Formatting
Tracking
(firefox122 fixed)
RESOLVED
FIXED
Tracking | Status | |
---|---|---|
firefox122 | --- | fixed |
People
(Reporter: scunnane, Assigned: standard8)
References
Details
Attachments
(2 files)
STR
- Open a YAML file and add trailing whitespace to a line in the file
- 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 | ||
Comment 1•6 months ago
|
||
Updated•6 months ago
|
Assignee: nobody → standard8
Status: NEW → ASSIGNED
Assignee | ||
Comment 2•6 months ago
|
||
Depends on D194918
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
Comment 4•6 months ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/1e3c5f46ea60
https://hg.mozilla.org/mozilla-central/rev/de47344d8f97
Status: ASSIGNED → RESOLVED
Closed: 6 months ago
status-firefox122:
--- → fixed
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•