path: add `path/posix` and `path/win32` alias modules #34962

ExE-Boss · 2020-08-28T23:11:31Z

Turns out, #34055 is not the last alias module, because I forgot about path.posix and path.win32.

Refs: #31553
Refs: #32953
Refs: #33950
Refs: #34001
Refs: #34002
Refs: #34055

Checklist

make -j4 test (UNIX), or vcbuild test (Windows) passes
tests and/or benchmarks are included
documentation is changed or added
commit message follows commit guidelines

Refs: nodejs#31553 Refs: nodejs#32953 Refs: nodejs#33950 Refs: nodejs#34001 Refs: nodejs#34002 Refs: nodejs#34055

MylesBorins

RSLGTM

ExE-Boss · 2020-10-12T17:15:47Z

review?(@jasnell, @mcollina, @targos, @tniessen, @Trott)

lib/path/posix.js

mcollina

lgtm

ExE-Boss · 2020-10-18T19:44:10Z

@jasnell @MylesBorins @Trott @targos @BridgeAR

It’s only a few days left until v15.0.0 ships, and I’d like to get this merged before then.

Trott · 2020-10-18T22:34:01Z

Repeating the comment from #34055 here as well:

We're a few weeks past the semver-major cut-off date for 15.x.x so this would probably require assent from both @nodejs/tsc and Releasers (and in particular @BethGriggs who is preparing the release) to land in 15.x.x.

@richardlau Is this semver major because it might have a naming clash with existing modules? Or is there another reason?

richardlau · 2020-10-18T22:41:40Z

Repeating the comment from #34055 here as well:

We're a few weeks past the semver-major cut-off date for 15.x.x so this would probably require assent from both @nodejs/tsc and Releasers (and in particular @BethGriggs who is preparing the release) to land in 15.x.x.

@richardlau Is this semver major because it might have a naming clash with existing modules? Or is there another reason?

I was under the impression that new modules were major due to naming clashes (and remember that "existing modules" include things that are not on npm's registry, e.g. local files). I think I did see someone post in an issue that that isn't what is writing down so 🤷.

BethGriggs · 2020-10-19T10:31:33Z

We're a few weeks past the semver-major cut-off date for 15.x.x so this would probably require assent from both @nodejs/tsc and Releasers (and in particular @BethGriggs who is preparing the release) to land in 15.x.x.

It would probably need to land in the next few hours to make it into v15.0.0 as I don't think we should make changes < 24 hours until the release (unless critical).

nodejs-github-bot · 2020-10-19T11:58:13Z

CI: https://ci.nodejs.org/job/node-test-pull-request/33718/

Trott · 2020-10-19T11:59:58Z

CITGM: https://ci.nodejs.org/view/Node.js-citgm/job/citgm-smoker/2525/

doc/api/path.md

Co-authored-by: Rich Trott <rtrott@gmail.com>

doc/api/path.md

Trott · 2020-10-19T13:02:05Z

I'm going to be offline for most of the day, but if someone else can try to shepherd this through, that would be great. CITGM spot check looks good to me, so it's really just a matter of resolving whatever lint issues might remain, getting a green CI, and landing.

Co-authored-by: Rich Trott <rtrott@gmail.com>

codecov-io · 2020-10-19T14:15:04Z

Codecov Report

Merging #34962 into master will decrease coverage by 0.47%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master   #34962      +/-   ##
==========================================
- Coverage   96.87%   96.40%   -0.48%     
==========================================
  Files         212      222      +10     
  Lines       69609    73681    +4072     
==========================================
+ Hits        67435    71033    +3598     
- Misses       2174     2648     +474

Impacted Files	Coverage Δ
lib/_http_client.js	`98.45% <100.00%> (+0.01%)`	⬆️
lib/assert.js	`98.09% <100.00%> (+<0.01%)`	⬆️
lib/events.js	`99.27% <100.00%> (+<0.01%)`	⬆️
lib/fs.js	`94.13% <100.00%> (-0.14%)`	⬇️
lib/internal/assert/assertion_error.js	`100.00% <100.00%> (ø)`
lib/internal/bootstrap/loaders.js	`97.18% <100.00%> (+0.01%)`	⬆️
lib/internal/buffer.js	`100.00% <100.00%> (ø)`
lib/internal/errors.js	`97.97% <100.00%> (+<0.01%)`	⬆️
lib/internal/event_target.js	`97.04% <100.00%> (+<0.01%)`	⬆️
lib/internal/fs/read_file_context.js	`98.31% <100.00%> (ø)`
... and 80 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8beef5e...c980bd8. Read the comment docs.

Refs: #31553 Refs: #32953 Refs: #33950 Refs: #34001 Refs: #34002 Refs: #34055 PR-URL: #34962 Reviewed-By: Myles Borins <myles.borins@gmail.com> Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de> Reviewed-By: Matteo Collina <matteo.collina@gmail.com> Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Gerhard Stöbich <deb2001-github@yahoo.de>

ljharb · 2020-10-20T16:04:44Z

v15 has already been cut before landing this, which means master is now v16?

Refs: #31553 Refs: #32953 Refs: #33950 Refs: #34001 Refs: #34002 PR-URL: #34055 Refs: #34962 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: Myles Borins <myles.borins@gmail.com> Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de> Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>

Fixes: nodejs#35740 Refs: nodejs#31553 Refs: nodejs#32953 Refs: nodejs#33991 Refs: nodejs#34001 Refs: nodejs#34055 Refs: nodejs#34962 Co-authored-by: Antoine du Hamel <duhamelantoine1995@gmail.com> Co-authored-by: Trivikram Kamat <trivikr.dev@gmail.com> Co-authored-by: Rich Trott <rtrott@gmail.com>

Fixes: #35740 Refs: #31553 Refs: #32953 Refs: #33991 Refs: #34001 Refs: #34055 Refs: #34962 Co-authored-by: Antoine du Hamel <duhamelantoine1995@gmail.com> Co-authored-by: Trivikram Kamat <trivikr.dev@gmail.com> Co-authored-by: Rich Trott <rtrott@gmail.com> PR-URL: #34002 Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Masashi Hirano <shisama07@gmail.com> Reviewed-By: Pranshu Srivastava <rexagod@gmail.com> Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>

Refs: #31553 Refs: #32953 Refs: #33950 Refs: #34001 Refs: #34002 Refs: #34055 PR-URL: #34962 Reviewed-By: Myles Borins <myles.borins@gmail.com> Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de> Reviewed-By: Matteo Collina <matteo.collina@gmail.com> Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Gerhard Stöbich <deb2001-github@yahoo.de>

Refs: #31553 Refs: #32953 Refs: #33950 Refs: #34001 Refs: #34002 PR-URL: #34055 Refs: #34962 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: Myles Borins <myles.borins@gmail.com> Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de> Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>

Notable changes: dns: * (SEMVER-MINOR) add a cancel() method to the promise Resolver (Szymon Marczak) #33099 events: * (SEMVER-MINOR) add max listener warning for EventTarget (James M Snell) #36001 http: * (SEMVER-MINOR) add support for abortsignal to http.request (Benjamin Gruenbaum) #36048 http2: * (SEMVER-MINOR) allow setting the local window size of a session (Yongsheng Zhang) #35978 lib: * (SEMVER-MINOR) add throws option to fs.f/l/statSync (Andrew Casey) #33716 path: * (SEMVER-MINOR) add `path/posix` and `path/win32` alias modules (ExE Boss) #34962 readline: * (SEMVER-MINOR) add getPrompt to get the current prompt (Mattias Runge-Broberg) #33675 src: * (SEMVER-MINOR) add loop idle time in diagnostic report (Gireesh Punathil) #35940 util: * (SEMVER-MINOR) add `util/types` alias module (ExE Boss) #34055 PR-URL: TODO

Notable changes: dns: * (SEMVER-MINOR) add a cancel() method to the promise Resolver (Szymon Marczak) #33099 events: * (SEMVER-MINOR) add max listener warning for EventTarget (James M Snell) #36001 http: * (SEMVER-MINOR) add support for abortsignal to http.request (Benjamin Gruenbaum) #36048 http2: * (SEMVER-MINOR) allow setting the local window size of a session (Yongsheng Zhang) #35978 lib: * (SEMVER-MINOR) add throws option to fs.f/l/statSync (Andrew Casey) #33716 path: * (SEMVER-MINOR) add `path/posix` and `path/win32` alias modules (ExE Boss) #34962 readline: * (SEMVER-MINOR) add getPrompt to get the current prompt (Mattias Runge-Broberg) #33675 src: * (SEMVER-MINOR) add loop idle time in diagnostic report (Gireesh Punathil) #35940 util: * (SEMVER-MINOR) add `util/types` alias module (ExE Boss) #34055 PR-URL: #36232

Fixes: #35740 Refs: #31553 Refs: #32953 Refs: #33991 Refs: #34001 Refs: #34055 Refs: #34962 Co-authored-by: Antoine du Hamel <duhamelantoine1995@gmail.com> Co-authored-by: Trivikram Kamat <trivikr.dev@gmail.com> Co-authored-by: Rich Trott <rtrott@gmail.com> PR-URL: #34002 Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Masashi Hirano <shisama07@gmail.com> Reviewed-By: Pranshu Srivastava <rexagod@gmail.com> Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>

nodejs-github-bot added the build label Aug 28, 2020

ExE-Boss mentioned this pull request Aug 28, 2020

util: add util/types alias module #34055

Closed

4 tasks

ExE-Boss mentioned this pull request Sep 29, 2020

2020-10-20, Version 15.0.0 (Current) #35014

Merged

richardlau added the semver-major label Sep 29, 2020

guybedford mentioned this pull request Oct 3, 2020

Node.js core alias modules jspm/jspm-core#11

Closed

ExE-Boss mentioned this pull request Oct 6, 2020

Node.js Technical Steering Committee (TSC) Meeting 2020-10-08 nodejs/TSC#935

Closed

ExE-Boss force-pushed the lib/path/add-path-posix-win-alias-modules branch from 440f191 to 4c75a27 Compare Oct 6, 2020

path: add path/posix and path/win32 alias modules

4c75a27

Refs: nodejs#31553 Refs: nodejs#32953 Refs: nodejs#33950 Refs: nodejs#34001 Refs: nodejs#34002 Refs: nodejs#34055

MylesBorins approved these changes Oct 8, 2020

View changes

BridgeAR approved these changes Oct 8, 2020

View changes

ljharb reviewed Oct 12, 2020

View changes

lib/path/posix.js Show resolved Hide resolved

mcollina approved these changes Oct 12, 2020

View changes

ExE-Boss requested a review from ljharb Oct 14, 2020

Trott approved these changes Oct 14, 2020

View changes

jasnell approved these changes Oct 14, 2020

View changes

Trott added the request-ci label Oct 19, 2020

github-actions bot removed the request-ci label Oct 19, 2020

Trott reviewed Oct 19, 2020

View changes

doc/api/path.md Outdated Show resolved Hide resolved

fixup! path: add path/posix and path/win32 alias modules

4635b04

Co-authored-by: Rich Trott <rtrott@gmail.com>

Trott reviewed Oct 19, 2020

View changes

doc/api/path.md Outdated Show resolved Hide resolved

fixup! path: add path/posix and path/win32 alias modules

c980bd8

Co-authored-by: Rich Trott <rtrott@gmail.com>

github-actions bot closed this Oct 20, 2020

ExE-Boss deleted the lib/path/add-path-posix-win-alias-modules branch Oct 20, 2020

ExE-Boss mentioned this pull request Oct 21, 2020

doc: document changes for */promises alias modules #34002

Closed

2 tasks

ExE-Boss mentioned this pull request Oct 24, 2020

Backporting util/types, path/posix and path/win32 to v15 proposal #35788

Closed

targos added semver-minor and removed semver-major labels Nov 11, 2020

codebytere mentioned this pull request Nov 22, 2020

v15.3.0 proposal #36232

Merged

targos added dont-land-on-v12.x dont-land-on-v14.x labels Aug 4, 2021

nodejs / node Public

path: add `path/posix` and `path/win32` alias modules #34962

path: add `path/posix` and `path/win32` alias modules #34962

ExE-Boss commented Aug 28, 2020 •

edited

MylesBorins left a comment

ExE-Boss commented Oct 12, 2020

mcollina left a comment

ExE-Boss commented Oct 18, 2020

Trott commented Oct 18, 2020

richardlau commented Oct 18, 2020

BethGriggs commented Oct 19, 2020

nodejs-github-bot commented Oct 19, 2020

Trott commented Oct 19, 2020

Trott commented Oct 19, 2020

codecov-io commented Oct 19, 2020

ljharb commented Oct 20, 2020

nodejs / node Public

path: add path/posix and path/win32 alias modules #34962

path: add path/posix and path/win32 alias modules #34962

Conversation

ExE-Boss commented Aug 28, 2020 • edited

Checklist

MylesBorins left a comment

ExE-Boss commented Oct 12, 2020

mcollina left a comment

ExE-Boss commented Oct 18, 2020

Trott commented Oct 18, 2020

richardlau commented Oct 18, 2020

BethGriggs commented Oct 19, 2020

nodejs-github-bot commented Oct 19, 2020

Trott commented Oct 19, 2020

Trott commented Oct 19, 2020

codecov-io commented Oct 19, 2020

Codecov Report

ljharb commented Oct 20, 2020

path: add `path/posix` and `path/win32` alias modules #34962

path: add `path/posix` and `path/win32` alias modules #34962

ExE-Boss commented Aug 28, 2020 •

edited