Closed
Bug 1240167
Opened 9 years ago
Closed 9 years ago
need eslint rule to mark top-level head.js definitions as used
Categories
(Testing :: General, defect)
Testing
General
Tracking
(Not tracked)
RESOLVED
WONTFIX
People
(Reporter: tromey, Unassigned)
References
Details
While trying to make a devtools directory eslint-clean, I noticed
many warnings coming from a test directory's head.js file.
In particular, many top-level definitions in this file trigger the
no-unused-var warning.
I think it would be good to have a special rule for head.js (and similar,
see bug 1240163) that marks the top-level definitions as used.
While it would be possible to disable the no-unused-var warning in these files,
it would be nicer to have a custom rule to minimize the damage.
Reporter | ||
Comment 1•9 years ago
|
||
One workaround is to use /* exports ... */ for each exported definition.
Maybe this is sufficient.
Reporter | ||
Comment 2•9 years ago
|
||
We discussed on irc and agreed that head.js files should just have:
/* eslint no-unused-vars: [2, {"vars": "local"}] */
So I think this bug is obsolete.
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•