1920531 - Remove html, json and prettier plugin dependencies from eslint-plugin-mozilla

Status: RESOLVED FIXED

(Developer Infrastructure :: Lint and Formatting, task, P3)

Description

[Mark Banner (:standard8)]

Separating this out from bug 1917529. For a few modules that are included as peer dependencies in eslint-plugin-mozilla, they seem unnecessary as:

• The html plugin is used as a parser and is only required if a project is parsing html.
• Similary, we use the recommended config for the JSON parser, and it only applies for JSON files.
• The prettier option is only required if a project wants to use prettier. Whilst I think we should have prettier for all Mozilla projects, I don't think we need to enforce/require it here.

Comment 1

[Mark Banner (:standard8)]

Attachment: Bug 1920531 - Remove html, json and prettier plugin dependencies from eslint-plugin-mozilla. r?#frontend-codestyle-reviewers!

Requiring these in eslint-plugin-mozilla seems a bit unnecessary:

• The html plugin is used as a parser and is only required if a project is parsing html.
• Similary, we use the recommended config for the JSON parser, and it only applies for JSON files.
• The prettier option is only required if a project wants to use prettier. Whilst I think we should have prettier for all Mozilla projects, I don't think we need to enforce/require it here.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/df73766cfce5
Remove html, json and prettier plugin dependencies from eslint-plugin-mozilla. r=frontend-codestyle-reviewers,Gijs

Comment 3

[Cristina Horotan [:chorotan]]

https://hg.mozilla.org/mozilla-central/rev/df73766cfce5