test,benchmark,doc: enable dot-notation rule

[BridgeAR]

This enables the eslint dot-notation rule for all code instead of
only in /lib.

Checklist

• make -j4 test (UNIX), or vcbuild test (Windows) passes
• tests and/or benchmarks are included
• documentation is changed or added
• commit message follows commit guidelines

Affected core subsystem(s)

test,doc,benchmark

[BridgeAR]

Light-CI https://ci.nodejs.org/job/node-test-commit-light/265/

[lpinca]

Yes, I was wrong. GitHub sends notifications too soon :)

…

On Tue, Feb 13, 2018 at 9:55 PM, Anatoli Papirovski < ***@***.***> wrote: ***@***.**** commented on this pull request. ------------------------------ In test/common/inspector-helper.js <#18749 (comment)>: > @@ -168,9 +168,7 @@ class InspectorSession{reject(message.error)} else{if (message.method === 'Debugger.scriptParsed'){- const script = message['params']; - const scriptId = script['scriptId']; - const url = script['url']; + const{scriptId, url } = message.params; I think it is, no? message['params']['scriptId'] and message['params']['url']. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#18749 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ABYIR2juHK9HRKgzN-vnit1RXpG2Q-Pqks5tUfbHgaJpZM4SDOjo> .

[BridgeAR]

CI before landing https://ci.nodejs.org/job/node-test-commit-light/271/

[BridgeAR]

Landed in 463d1a4