Uh oh!
There was an error while loading. Please reload this page.
test: update test-cluster-worker-events to use arrow functions. - #23469
test: update test-cluster-worker-events to use arrow functions.#23469Ev-Abbott wants to merge 3 commits into
Conversation
digitalinfinity
commented
Oct 12, 2018
@AbbottEverett thanks for your PR- the change itself looks good to me but your git config needs a little tweak. Can you run the following command to set your email in your git config: Then run (thanks to @gireeshpunthil for this tip!) |
Ev-Abbott
commented
Oct 12, 2018
Thanks for the advice @digitalinfinity. I made the changes and pushed to this branch. |
| const messages = []; | ||
| function check(m) { | ||
| const check = (m) => { |
There was a problem hiding this comment.
Just a nit: for top level functions like this, using the function check(m) { form is preferred. The arrow functions on the event handlers is fine.
Trott
commented
Oct 13, 2018
Trott
commented
Oct 13, 2018
Collaborators, please 👍 here to fast-track. |
Trott
commented
Oct 13, 2018
Landed in 3989785. Thanks for the contribution! 🎉 (If you're interested in other possible contributions to Node.js but don't have a good idea of where to start looking, some ideas are posted at https://www.nodetodo.org/next-steps/.) |
PR-URL: nodejs#23469 Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com> Reviewed-By: James M Snell <jasnell@gmail.com>
PR-URL: #23469 Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com> Reviewed-By: James M Snell <jasnell@gmail.com>
PR-URL: #23469 Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com> Reviewed-By: James M Snell <jasnell@gmail.com>
PR-URL: #23469 Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com> Reviewed-By: James M Snell <jasnell@gmail.com>
PR-URL: #23469 Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com> Reviewed-By: James M Snell <jasnell@gmail.com>
PR-URL: #23469 Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com> Reviewed-By: James M Snell <jasnell@gmail.com>
Checklist
make -j4 test(UNIX), orvcbuild test(Windows) passes