-
Notifications
You must be signed in to change notification settings - Fork 29.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
doc: fix typo in events.getEventListeners
example
#36085
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks good catch!
API feedback welcome as well :]
If you want to find more places to contribute - this API could probably use a few more tests as well!
As this is a docs typo and a first time contributor: 👍 to fast track |
This comment has been minimized.
This comment has been minimized.
Commit Queue failed- Loading data for nodejs/node/pull/36085 ✔ Done loading data for nodejs/node/pull/36085 ----------------------------------- PR info ------------------------------------ Title doc: fix typo in `events.getEventListeners` example (#36085) Author Dmitry Semigradsky (@Semigradsky, first-time contributor) Branch Semigradsky:patch-1 -> nodejs:master Labels author ready, doc, events Commits 1 - doc: fix `events.getEventListeners` example Committers 1 - GitHub PR-URL: https://github.com/nodejs/node/pull/36085 Reviewed-By: Luigi Pinca Reviewed-By: Benjamin Gruenbaum Reviewed-By: Yongsheng Zhang Reviewed-By: Antoine du Hamel ------------------------------ Generated metadata ------------------------------ PR-URL: https://github.com/nodejs/node/pull/36085 Reviewed-By: Luigi Pinca Reviewed-By: Benjamin Gruenbaum Reviewed-By: Yongsheng Zhang Reviewed-By: Antoine du Hamel -------------------------------------------------------------------------------- ✔ Last GitHub Actions successful ℹ Doc-only changes ℹ This PR was created on Wed, 11 Nov 2020 15:54:59 GMT ✔ Approvals: 4 ✔ - Luigi Pinca (@lpinca): https://github.com/nodejs/node/pull/36085#pullrequestreview-528307178 ✔ - Benjamin Gruenbaum (@benjamingr): https://github.com/nodejs/node/pull/36085#pullrequestreview-528329331 ✔ - Yongsheng Zhang (@ZYSzys): https://github.com/nodejs/node/pull/36085#pullrequestreview-528351605 ✔ - Antoine du Hamel (@aduh95): https://github.com/nodejs/node/pull/36085#pullrequestreview-528412376 ✖ This PR needs to wait 45 more hours to land -------------------------------------------------------------------------------- ✔ Aborted `git node land` session in /home/runner/work/node/node/.ncu Commit Queue action: https://github.com/nodejs/node/actions/runs/358244318 |
Landed in c0ac692...a8b8b8b |
PR-URL: #36085 Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Benjamin Gruenbaum <[email protected]> Reviewed-By: Yongsheng Zhang <[email protected]> Reviewed-By: Antoine du Hamel <[email protected]>
Congrats on your first contribution @Semigradsky 🎉 and a personal thanks for fixing my bug! |
PR-URL: #36085 Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Benjamin Gruenbaum <[email protected]> Reviewed-By: Yongsheng Zhang <[email protected]> Reviewed-By: Antoine du Hamel <[email protected]>
Checklist
make -j4 test
(UNIX), orvcbuild test
(Windows) passes