1395176 - Add eslint rule to forbid use of var, except at top-level

Status: RESOLVED FIXED

(Remote Protocol :: Marionette, enhancement)

Comment 1

[Andreas Tolfsen ❲:ato❳]

Attachment: Bug 1395176 - Use scoped remoteFrames instead of global.

Review commit: https://reviewboard.mozilla.org/r/174416/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/174416/

Comment 2

[Andreas Tolfsen ❲:ato❳]

Attachment: Bug 1395176 - Turn marshal helper into function.

Review commit: https://reviewboard.mozilla.org/r/174418/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/174418/

Comment 3

[Andreas Tolfsen ❲:ato❳]

Attachment: Bug 1395176 - Lint testing/marionette for var usage.

Review commit: https://reviewboard.mozilla.org/r/174420/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/174420/

Comment 4

[Andreas Tolfsen ❲:ato❳]

Attachment: Bug 1395176 - Add eslint rule that forbids use of var.

Review commit: https://reviewboard.mozilla.org/r/174422/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/174422/

Comment 5

[David Burns :automatedtester]

Comment on attachment 8902757 [details]
Bug 1395176 - Use scoped remoteFrames instead of global.

https://reviewboard.mozilla.org/r/174416/#review179714

Comment 6

[David Burns :automatedtester]

Comment on attachment 8902758 [details]
Bug 1395176 - Turn marshal helper into function.

https://reviewboard.mozilla.org/r/174418/#review179718

Comment 7

[David Burns :automatedtester]

Comment on attachment 8902759 [details]
Bug 1395176 - Lint testing/marionette for var usage.

https://reviewboard.mozilla.org/r/174420/#review179728

Comment 8

[David Burns :automatedtester]

Comment on attachment 8902760 [details]
Bug 1395176 - Add eslint rule that forbids use of var.

https://reviewboard.mozilla.org/r/174422/#review179730

Comment 9

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/5fcf97b0946a
Use scoped remoteFrames instead of global. r=automatedtester
https://hg.mozilla.org/integration/autoland/rev/db4255debad2
Turn marshal helper into function. r=automatedtester
https://hg.mozilla.org/integration/autoland/rev/a9deb69ab381
Lint testing/marionette for var usage. r=automatedtester
https://hg.mozilla.org/integration/autoland/rev/4b2c8d6a06b6
Add eslint rule that forbids use of var. r=automatedtester

Comment 10

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

https://hg.mozilla.org/mozilla-central/rev/5fcf97b0946a
https://hg.mozilla.org/mozilla-central/rev/db4255debad2
https://hg.mozilla.org/mozilla-central/rev/a9deb69ab381
https://hg.mozilla.org/mozilla-central/rev/4b2c8d6a06b6