-
Notifications
You must be signed in to change notification settings - Fork 29.7k
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 a typo in CHANGELOG_V15 #35804
Conversation
Since the typo is in the commit message, this will mean that the change log will be different from the commit log. In the past, we've opted to leave typographical errors unfixed in the change log if the error is derived from commit messages via tooling, like this one. See, for example, #20716 (comment). (I don't have strong feelings either way. I'm fine with fixing them and I'm fine with leaving them alone.) |
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.
Nice catch! Thank youuu!
Signed-off-by: Takuya Noguchi <takninnovationresearch@gmail.com> PR-URL: nodejs#35804 Reviewed-By: Franziska Hinkelmann <franziska.hinkelmann@gmail.com> Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By: Daijiro Wachi <daijiro.wachi@gmail.com> Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com> Reviewed-By: Juan José Arboleda <soyjuanarbol@gmail.com>
ecb432e
to
02e60a6
Compare
Landed in 02e60a6 Thanks for the contribution! 🎉 |
Signed-off-by: Takuya Noguchi <takninnovationresearch@gmail.com> PR-URL: #35804 Reviewed-By: Franziska Hinkelmann <franziska.hinkelmann@gmail.com> Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By: Daijiro Wachi <daijiro.wachi@gmail.com> Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com> Reviewed-By: Juan José Arboleda <soyjuanarbol@gmail.com>
Fix a typo around #32984.
Checklist