benchmark: use let instead of var in assert#30450
benchmark: use let instead of var in assert#30450dnlup wants to merge 1 commit intonodejs:masterfrom
Conversation
585c821 to
350075c
Compare
350075c to
294c7d6
Compare
|
Going slightly out of scope. |
294c7d6 to
769540a
Compare
@SimonSchick Short answer is most likely "No" This is one of the Code and Learn tasks from workshop at NodeConfEU:
There are many other PRs which are doing var to let/const replacements in other files.
I don't recommend doing it for all files at once it'l be difficult to review. For example, we still use
I think you can replace vs to let/const on module basis (one PR for http, one for fs etc) so that only the particular module team can review it. |
PR-URL: #30450 Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: David Carlier <devnexen@gmail.com> Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com> Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de> Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
|
landed in d1ce04c , thanks for the contribution! |
PR-URL: #30450 Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: David Carlier <devnexen@gmail.com> Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com> Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de> Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
PR-URL: #30450 Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: David Carlier <devnexen@gmail.com> Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com> Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de> Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
PR-URL: #30450 Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: David Carlier <devnexen@gmail.com> Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com> Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de> Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Use let instead of var in benchmark/assert/deepequal-buffer.js.
Checklist
make -j4 test(UNIX), orvcbuild test(Windows) passes