-
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
lib: fix typos in lib code comments #40792
Conversation
As per the review, welcome @kuriyosh! And thanks for the pull request. Good catches there, did you find these typos by eyesight or by the help of a tool? If a tool; then which? It would be helpful for us to use these kinds of tools to find typos. |
@VoltrexMaster I used aspell which is a spell checker tool with a custom dictionary registered some words used in |
Fast-track has been requested by @Ayase-252. Please 👍 to approve. |
@watilde we shouldn't run full Jenkins CIs on changes that only apply on the documentation or comment changes as it does not affect the node binary builds, but one has already ran so; might need to remember that. 😄 |
Landed in 69f487e, thanks a lot for the contribution 🎉 |
PR-URL: #40792 Reviewed-By: Voltrex <mohammadkeyvanzade94@gmail.com> Reviewed-By: Qingyu Deng <i@ayase-lab.com> Reviewed-By: Daijiro Wachi <daijiro.wachi@gmail.com> Reviewed-By: Tobias Nießen <tniessen@tnie.de> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
PR-URL: #40792 Reviewed-By: Voltrex <mohammadkeyvanzade94@gmail.com> Reviewed-By: Qingyu Deng <i@ayase-lab.com> Reviewed-By: Daijiro Wachi <daijiro.wachi@gmail.com> Reviewed-By: Tobias Nießen <tniessen@tnie.de> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
PR-URL: #40792 Reviewed-By: Voltrex <mohammadkeyvanzade94@gmail.com> Reviewed-By: Qingyu Deng <i@ayase-lab.com> Reviewed-By: Daijiro Wachi <daijiro.wachi@gmail.com> Reviewed-By: Tobias Nießen <tniessen@tnie.de> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
This fixes three typos in lib dir code comment.
methdod
→method
inlib/_http_incoming.js
catcheable
→catchable
inlib/events.js
instancee
→instance
inlib/path.js