-
Notifications
You must be signed in to change notification settings - Fork 30.2k
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: 96 files have "the the" typo #20682
Comments
All the occurrences in All the occurrences in That leaves five occurrences in |
$ grep -r 'the the' --exclude-dir=deps --exclude-dir=out --exclude-dir=node_modules | wc -l
12
$ grep -r 'the the ' --exclude-dir=deps --exclude-dir=out --exclude-dir=node_modules | wc -l
5 Full output for the remaining ones: $ grep -r 'the the ' --exclude-dir=deps --exclude-dir=out --exclude-dir=node_modules
tools/msvs/msi/i18n/en-us.wxl: <String Id="DocumentationShortcuts_Description">Add start menu entries that link the the online documentation for [ProductName] [FullVersion] and the [ProductName] website.</String>
tools/gyp/pylib/gyp/MSVSSettings.py: """Convert the the MSVS macros found in the string to the MSBuild equivalent.
tools/gyp/pylib/gyp/generator/make.py:# the side to keep the the files readable.
tools/gyp/pylib/gyp/generator/ninja.py: # so that the the target that directly depends on the .objs can also depend
doc/changelogs/CHANGELOG_V8.md:* [[`816f98f5d0`](https://github.com/nodejs/node/commit/816f98f5d0)] - **console**: use a plain object for the the error stack (Ruben Bridgewater) [#13743](https://github.com/nodejs/node/pull/13743) Upd: ah, @Trott beat me to it by seconds. 😆 |
I had spotted a few in the past, which I saved and leveraged for a code & learn activity. :) It is possible that this is a common typo pattern that recur. |
Fix typo `the the`. Fixes: nodejs#20682
PR-URL: #20716 Fixes: #20682 Reviewed-By: Trivikram Kamat <[email protected]> Reviewed-By: Ruben Bridgewater <[email protected]> Reviewed-By: Michael Dawson <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: James M Snell <[email protected]>
PR-URL: nodejs/node#20716 Fixes: nodejs/node#20682 Reviewed-By: Trivikram Kamat <[email protected]> Reviewed-By: Ruben Bridgewater <[email protected]> Reviewed-By: Michael Dawson <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: James M Snell <[email protected]>
PR-URL: nodejs/node#20716 Fixes: nodejs/node#20682 Reviewed-By: Trivikram Kamat <[email protected]> Reviewed-By: Ruben Bridgewater <[email protected]> Reviewed-By: Michael Dawson <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: James M Snell <[email protected]>
PR-URL: nodejs#20716 Fixes: nodejs#20682 Reviewed-By: Trivikram Kamat <[email protected]> Reviewed-By: Ruben Bridgewater <[email protected]> Reviewed-By: Michael Dawson <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: James M Snell <[email protected]> (cherry picked from commit 3b1bf0d)
There are 96 files including a common typo of "the the" in this repository, to be precise, in
deps
,doc
, andtools
directory. I assume some of them are 3rd party's and no need to revise, but is there anything that needs to be fixed?Full output
The text was updated successfully, but these errors were encountered: