1240167 - need eslint rule to mark top-level head.js definitions as used

Status: RESOLVED WONTFIX

(Testing :: General, defect)

Description

[Tom Tromey :tromey]

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.

Comment 1

[Tom Tromey :tromey]

One workaround is to use /* exports ... */ for each exported definition.
Maybe this is sufficient.

Comment 2

[Tom Tromey :tromey]

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.