os: implement os.type() using uv_os_uname() by cjihrig · Pull Request #25659 · nodejs/node (original) (raw)

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 andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Conversation2 Commits1 Checks0 Files changed

Conversation

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.Learn more about bidirectional Unicode characters

[ Show hidden characters]({{ revealButtonHref }})

cjihrig

The happy path behavior should be identical on all platforms except MinGW, which now identifies MinGW separately from Windows.

Labeling this as semver-major due to the change in behavior on MinGW. I could also #ifdef the previous behavior on MinGW, and remove it in a later semver-major commit.

Checklist

@cjihrig cjihrig added the semver-major

PRs that contain breaking changes and should be released in the next major version.

label

Jan 23, 2019

@nodejs-github-bot

@nodejs-github-bot nodejs-github-bot added c++

Issues and PRs that require attention from people who are familiar with C++.

os

Issues and PRs related to the os subsystem.

labels

Jan 23, 2019

bzoz

bzoz approved these changes Jan 23, 2019

addaleax

JungMinu

Fishrock123

@cjihrig

@addaleax addaleax added the author ready

PRs that have at least one approval, no pending requests for changes, and a CI started.

label

Jan 24, 2019

@cjihrig

The happy path behavior should be identical on all platforms except MinGW, which now identifies MinGW separately from Windows.

PR-URL: nodejs#25659 Reviewed-By: Bartosz Sosnowski bartosz@janeasystems.com Reviewed-By: Anna Henningsen anna@addaleax.net Reviewed-By: Minwoo Jung minwoo@nodesource.com Reviewed-By: Jeremiah Senkpiel fishrock123@rocketmail.com

BethGriggs added a commit that referenced this pull request

Apr 22, 2019

@BethGriggs

Notable changes:

PR-URL: #26930

BethGriggs added a commit that referenced this pull request

Apr 23, 2019

@BethGriggs

Notable changes:

PR-URL: #26930

BethGriggs added a commit that referenced this pull request

Apr 23, 2019

@BethGriggs

Notable changes:

PR-URL: #26930

This was referenced

Apr 23, 2019

Labels

author ready

PRs that have at least one approval, no pending requests for changes, and a CI started.

c++

Issues and PRs that require attention from people who are familiar with C++.

os

Issues and PRs related to the os subsystem.

semver-major

PRs that contain breaking changes and should be released in the next major version.