Andreas Tolfsen ❲:ato❳ Assignee
	Description • 7 years ago

      No description provided.

	Henrik Skupin [:whimboo][⌚️UTC+1]
	Comment 2 • 7 years ago
mozreview-review

Comment on attachment 8914369 [details]
Bug 1405004 - Remove unused imports from testing/marionette/element.js.

https://reviewboard.mozilla.org/r/185640/#review190756

I wonder if we could lint for those things.

	Pulsebot
	Comment 3 • 7 years ago

Pushed by atolfsen@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/557a4ad274ed
Remove unused imports from testing/marionette/element.js. r=whimboo

	Andreas Tolfsen ❲:ato❳ Assignee
	Comment 4 • 7 years ago

(In reply to Henrik Skupin (:whimboo) from comment #2)

> I wonder if we could lint for those things.

That would be nice, but in an interpreted language I guess that
would be hard.  Maybe https://eslint.org/docs/rules/no-unused-vars
would be helpful?  I’m not sure how good it is.

	Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)
	Comment 5 • 7 years ago
bugherder

https://hg.mozilla.org/mozilla-central/rev/557a4ad274ed

	Andreas Tolfsen ❲:ato❳ Assignee
	Comment 6 • 7 years ago

(In reply to Andreas Tolfsen ‹:ato› from comment #4)
> (In reply to Henrik Skupin (:whimboo) from comment #2)
> 
> > I wonder if we could lint for those things.
> 
> That would be nice, but in an interpreted language I guess that
> would be hard.  Maybe https://eslint.org/docs/rules/no-unused-vars
> would be helpful?  I’m not sure how good it is.

Filed https://bugzil.la/1405279 to follow up on this.