benchmark,doc,lib,test: capitalize comments#26223
benchmark,doc,lib,test: capitalize comments#26223BridgeAR wants to merge 3 commits intonodejs:masterfrom
Conversation
This updates a lot of comments.
Trott
left a comment
There was a problem hiding this comment.
RSLGTM, but are you willing to backport to the four current release branches (11, 10, 8, 6) so as to avoid endless merge conflicts going forward?
(I guess if churn is a concern, we can always try 45 or 49 instead of 40?)
|
@Trott yes, I am willing to do that. So far it seemed like the introduction of the rule itself did not cause any pain in backporting. It's probably because it's mostly just a single line in a comment and this can mostly be resolved automatically. |
vsemozhetbyt
left a comment
There was a problem hiding this comment.
With in-passing nits, feel free to ignore or to address in a fixup commit/other PR.
Co-Authored-By: BridgeAR <ruben@bridgewater.de>
Co-Authored-By: BridgeAR <ruben@bridgewater.de>
antsmartian
left a comment
There was a problem hiding this comment.
Thanks @BridgeAR 👍 👍
This updates a lot of comments. PR-URL: nodejs#26223 Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com> Reviewed-By: Anto Aravinth <anto.aravinth.cse@gmail.com>
|
Landed in 9edce1e |
|
This needs a manual backport for v11.x, which should probably happen soon, given that this changed 200 files. It might also make sense to not land the bits that change the linter rules for the release branches, because that can get in the way of backporting this? (Maybe it could have been a separate commit? I know other people do it that way when modifying/adding/enabling linter rules.) |
This updates a lot of comments. PR-URL: #26223 Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com> Reviewed-By: Anto Aravinth <anto.aravinth.cse@gmail.com>
|
I backported this directly to the staging branch. @addaleax I'll separate the PRs in the future. That is indeed a good way of dealing with that. In this case I just ran |
This updates a lot of comments.
Checklist
make -j4 test(UNIX), orvcbuild test(Windows) passes