Enable the no-var rule in the src/core/evaluator.js file
#13344
Conversation
These changes were made automatically, using `gulp lint --fix`.
…tically, in the `src/core/evaluator.js` file The only *slight* complication here were some of the `switch`-cases, in `getOperatorList`/`getTextContent`, where the parsing is done asynchronously. However, those cases are easy to deal with by wrapping the code within its own block; please see https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Statements/switch#block-scope_variables_within_switch_statements
|
This wasn't as difficult as I previously thought, probably thanks to more experience than last time I looked at this :-) /botio test |
From: Bot.io (Linux m4)ReceivedCommand cmd_test from @Snuffleupagus received. Current queue size: 0 Live output at: http://54.67.70.0:8877/a90256270cae395/output.txt |
From: Bot.io (Windows)ReceivedCommand cmd_test from @Snuffleupagus received. Current queue size: 0 Live output at: http://3.101.106.178:8877/b2920778fb746b7/output.txt |
From: Bot.io (Linux m4)FailedFull output at http://54.67.70.0:8877/a90256270cae395/output.txt Total script time: 25.93 mins
Image differences available at: http://54.67.70.0:8877/a90256270cae395/reftest-analyzer.html#web=eq.log |
From: Bot.io (Windows)FailedFull output at http://3.101.106.178:8877/b2920778fb746b7/output.txt Total script time: 29.69 mins
Image differences available at: http://3.101.106.178:8877/b2920778fb746b7/reftest-analyzer.html#web=eq.log |
|
Thanks! |
The only slight complication here were some of the
switch-cases, ingetOperatorList/getTextContent, where the parsing is done asynchronously.However, those cases are easy to deal with by wrapping the code within its own block; please see https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Statements/switch#block-scope_variables_within_switch_statements