Closed
Bug 1830265
Opened 1 year ago
Closed 1 year 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•1 year ago
|
||
|
|
Assignee | |
Comment 2•1 year ago
|
||
Depends on D176634
|
|
Assignee | |
Comment 3•1 year ago
|
||
Depends on D176635
|
|
Assignee | |
Comment 4•1 year 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•1 year ago
|
||
| bugherder | ||
You need to log in
before you can comment on or make changes to this bug.
Description
•