Closed
Bug 1830265
Opened 2 years ago
Closed 2 years ago
Tidy up the start of .prettierignore
Categories
(Developer Infrastructure :: Lint and Formatting, task, P2)
Developer Infrastructure
Lint and Formatting
Tracking
(firefox114 fixed)
RESOLVED
FIXED
| Tracking | Status | |
|---|---|---|
| firefox114 | --- | fixed |
People
(Reporter: standard8, Assigned: standard8)
References
Details
Attachments
(4 files)
The structure of the first part of .prettierignore is a bit varied, and as we're going to be adding more items to it soon, I'd like to tidy it up a bit.
There's a few preference files that I don't think need to be excluded from Prettier (and ESLint). They aren't pre-formatted, so we might as well make them consistent with the rest of our JavaScript.
There's also a few sections where they are now duplicates of the .eslintignore parts that have been included, hence we can remove those as well, as even when we remove the duplication (bug 1825508), we'll keep Prettier loading .eslintignore.
|
Assignee | |
Comment 1•2 years ago
|
||
|
|
Assignee | |
Comment 2•2 years ago
|
||
Depends on D176634
|
|
Assignee | |
Comment 3•2 years ago
|
||
Depends on D176635
|
|
Assignee | |
Comment 4•2 years ago
|
||
Depends on D176636
Pushed by mbanner@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/f6d138a3b18a
Enable Prettier and ESLint on toolkit/components/backgroundtasks/defaults. r=nalexander
https://hg.mozilla.org/integration/autoland/rev/546391a0de46
Enable prettier formatting on testing/profiles/**/user.js files. r=ahal,perftest-reviewers,sparky
https://hg.mozilla.org/integration/autoland/rev/54d977d8f575
Tidy up .prettierignore. r=Gijs,sync-reviewers,markh
https://hg.mozilla.org/integration/autoland/rev/16a5fe74d586
Enable ESLint and Prettier on .babel-eslint.rc.js. r=Gijs
|
||
Comment 6•2 years ago
|
||
| bugherder | ||
You need to log in
before you can comment on or make changes to this bug.
Description
•