Need better README/docs for npm for eslint-plugin-mozilla

RESOLVED FIXED in Firefox 55

Status

RESOLVED FIXED
2 years ago
9 months ago

People

(Reporter: pdehaan, Assigned: standard8)

Tracking

unspecified
mozilla55

Firefox Tracking Flags

(firefox55 fixed)

Details

Attachments

(1 attachment)

(Reporter)

Description

2 years ago
Recently it looks like something changed and broke our ESLint (see https://github.com/mozilla/activity-stream/issues/1687).

Looking at the https://www.npmjs.com/package/eslint-plugin-mozilla page doesn't give any real useful information (like repo, rules, etc).

https://pageshot.net/46rrU7wdMfFiSOR3/www.npmjs.com

It'd be nice if there was a public README or something on npm so it's easier to see what's changed and what the current rules are.
Additionally, the latest readthedocs seems to imply the rules for "components-imports" etc still exist:

http://gecko.readthedocs.io/en/latest/tools/lint/linters/eslint-plugin-mozilla.html
Component: Developer Tools → Lint
Product: Firefox → Testing
A related issue I guess is that there is no standard process for publishing update to npm.  Inside of m-c, the source is used directly as-is and npm is not used for the Mozilla plugin, but external process would (I assume) want changes via npm.
Comment hidden (mozreview-request)
(Assignee)

Updated

2 years ago
Blocks: 1305527
(Assignee)

Comment 4

2 years ago
I've added a readme & package.json in the patch. In bug 1305527 I'm going to look at getting it updated, and then work out about regular updates later.
Assignee: nobody → standard8

Comment 5

2 years ago
mozreview-review
Comment on attachment 8845368 [details]
Bug 1314405 - Provide a readme for eslint-plugin-mozilla, and add some more details to its package.json.

https://reviewboard.mozilla.org/r/118570/#review120612

::: tools/lint/eslint/eslint-plugin-mozilla/README.md:27
(Diff revision 1)
> +```
> +
> +Next, install `eslint-plugin-mozilla`:
> +
> +```
> +$ npm install eslint-plugin-mozilla --save-dev

Do we actually publish this to npm?
(Assignee)

Comment 6

2 years ago
(In reply to Dave Townsend [:mossop] from comment #5)
> ::: tools/lint/eslint/eslint-plugin-mozilla/README.md:27
> (Diff revision 1)
> > +```
> > +
> > +Next, install `eslint-plugin-mozilla`:
> > +
> > +```
> > +$ npm install eslint-plugin-mozilla --save-dev
> 
> Do we actually publish this to npm?

Yes: https://www.npmjs.com/package/eslint-plugin-mozilla

It is a little out of date, but I'm going to be updating it. Devtools uses it for a few things, and I'm looking at using it for the one-off-system-addons repo.

Comment 7

2 years ago
mozreview-review
Comment on attachment 8845368 [details]
Bug 1314405 - Provide a readme for eslint-plugin-mozilla, and add some more details to its package.json.

https://reviewboard.mozilla.org/r/118570/#review120620
Attachment #8845368 - Flags: review?(dtownsend) → review+

Comment 8

2 years ago
Pushed by mbanner@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/00beed695243
Provide a readme for eslint-plugin-mozilla, and add some more details to its package.json. r=mossop

Comment 9

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/00beed695243
Status: NEW → RESOLVED
Last Resolved: 2 years ago
status-firefox55: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla55

Updated

9 months ago
Product: Testing → Firefox Build System
You need to log in before you can comment on or make changes to this bug.