-
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
Should we replace the word "OS X" with "macOS"? #12086
Comments
I don't think it is a good idea to change that in the source code, but the documentation could be updated. BUILDING.md77:### Unix / OS X 86:On OS X, you will also need: 240:##### Unix / OS X: 257:##### Unix / OS X: 269:#### Use existing installed ICU (Unix / OS X only): 285:##### Unix / OS X CONTRIBUTING.md160:To run the tests on Unix / OS X: doc/api/child_process.md436:*Note: Certain platforms (OS X, Linux) will use the value of `argv[0]` for the doc/api/documentation.md76:example, that `fs.lchown()` only works on Mac OS X and other BSD-derived systems, 80:relative to Linux and OS X. For an example of the subtle ways in which it's doc/api/errors.md526: parallel, especially on systems (in particular, OS X) where there is a low doc/api/fs.md1102:Only available on Mac OS X. 1377:opening a directory on OS X and Linux with the `'a+'` flag - see example below - 1382:// OS X and Linux 1946:The recursive option is only supported on OS X and Windows. 1957:* On OS X, this uses [`kqueue`] for files and [`FSEvents`] for directories. 1975:On Linux and OS X systems, `fs.watch()` resolves the path to an [inode][] and 1985:lifetime of a file, that way though this is different from Linux / OS X, doc/api/net.md28:The typical values are 107 on Linux and 103 on OS X. The path is doc/api/os.md366:as returned by uname(3). For example `'Linux'` on Linux, `'Darwin'` on OS X and doc/api/process.md1628:limited. For instance, on Linux and OS X, `process.title` is limited to the size doc/releases.md191:This is particularly recommended if there has been recent work relating to the OS X or Windows installers as they are not tested in any way by CI. 197:Use this opportunity particularly to test OS X and Windows installers if there are any concerns. 207:It is only necessary to have the main Linux (x64 and x86), OS X .pkg and .tar.gz, Windows (x64 and x86) .msi and .exe, source, headers and docs (both produced currently by an OS X slave). Note: it might be a good opportunity to change the word "slave" there to something else too. test/README.md295:Platform check for OS X. doc/api/child_process.md78:systems (Unix, Linux, OSX) [`child_process.execFile()`][] can be more efficient doc/STYLE_GUIDE.md42:* When using dashes, use emdashes ("—", Ctrl+Alt+"-" on OSX) surrounded by |
I think source code comment could be updated too |
@andmeics IMO, there's no point in littering the Git history (i.e., breaking |
I know,do it can help docs unified |
@andmeics would you be interested in opening a PR for that? :) |
Yeah let's just leave the source code. People will know what it means for years. IMO we should definitely update the docs though. |
This commit updates two paragraphs that exceeded the 80 line standard after updating to macOS. PR-URL: #12106 Fixes: #12086 Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Richard Lau <riclau@uk.ibm.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
This commit updates 3 additional references to Mac OS X in releases.md to macOS. PR-URL: #12106 Fixes: #12086 Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Richard Lau <riclau@uk.ibm.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
This commit updates 3 additional references to Mac OS X in releases.md to macOS. PR-URL: nodejs#12106 Fixes: nodejs#12086 Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Richard Lau <riclau@uk.ibm.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
This update changes references to "Mac OS X", "OS X", and "OSX" in markdown files to "macOS". Fixes: #12086 PR-URL: #12106 Backport-PR-URL: #12650 Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Richard Lau <riclau@uk.ibm.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
This commit updates two paragraphs that exceeded the 80 line standard after updating to macOS. Fixes: #12086 PR-URL: #12106 Backport-PR-URL: #12650 Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Richard Lau <riclau@uk.ibm.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
This commit updates 3 additional references to Mac OS X in releases.md to macOS. Fixes: #12086 PR-URL: #12106 Backport-PR-URL: #12650 Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Richard Lau <riclau@uk.ibm.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
This update changes references to "Mac OS X", "OS X", and "OSX" in markdown files to "macOS". Fixes: #12086 PR-URL: #12106 Backport-PR-URL: #12650 Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Richard Lau <riclau@uk.ibm.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
This commit updates two paragraphs that exceeded the 80 line standard after updating to macOS. Fixes: #12086 PR-URL: #12106 Backport-PR-URL: #12650 Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Richard Lau <riclau@uk.ibm.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
This commit updates 3 additional references to Mac OS X in releases.md to macOS. Fixes: #12086 PR-URL: #12106 Backport-PR-URL: #12650 Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Richard Lau <riclau@uk.ibm.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
This update changes references to "Mac OS X", "OS X", and "OSX" in markdown files to "macOS". Fixes: nodejs/node#12086 PR-URL: nodejs/node#12106 Backport-PR-URL: nodejs/node#12650 Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Richard Lau <riclau@uk.ibm.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
This commit updates two paragraphs that exceeded the 80 line standard after updating to macOS. Fixes: nodejs/node#12086 PR-URL: nodejs/node#12106 Backport-PR-URL: nodejs/node#12650 Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Richard Lau <riclau@uk.ibm.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
This commit updates 3 additional references to Mac OS X in releases.md to macOS. Fixes: nodejs/node#12086 PR-URL: nodejs/node#12106 Backport-PR-URL: nodejs/node#12650 Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Richard Lau <riclau@uk.ibm.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
Last year,Apple update MacOS X 10.11 to macOS Sierra(10.12),so "OSX" "OS X" and "MACOSX" more and less are old.Should we update "OSX" "OS X" and "MACOSX" into "macOS" in our code?
The text was updated successfully, but these errors were encountered: