GeezerVison

The World Wide Web, restored to 800 pixels of glory.

GeezerVison is showing github.com — v19.5.0 proposal by RafaelGSS · Pull Request #46286 · nodejs/node

Conversation

RafaelGSS commented Jan 20, 2023 •

edited

Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link

Copy Markdown

Member

2023-01-24, Version 19.5.0 (Current), @RafaelGSS

Notable Changes

http:
- (SEMVER-MINOR) join authorization headers (Marco Ippolito) #45982
lib::
- add webstreams to Duplex.from() (Debadree Chatterjee) #46190
stream:
- implement finished() for ReadableStream and WritableStream (Debadree Chatterjee) #46205

Commits

[def36946da] - assert: remove assert.snapshot (Moshe Atlow) #46112
[e1c56ec3fd] - benchmark,tools: use os.availableParallelism() (Deokjin Kim) #46003
[370f621d4d] - build: add extra semi check (Jiawen Geng) #46194
[476c6f892d] - crypto: avoid hang when no algorithm available (Richard Lau) #46237
[8b22310940] - (SEMVER-MINOR) crypto: add CryptoKey Symbol.toStringTag (Filip Skokan) #46042
[78be87b9f9] - crypto: add cipher update/final methods encoding validation (vitpavlenko) #45990
[dc0cdaa101] - crypto: ensure auth tag set for chacha20-poly1305 (Ben Noordhuis) #46185
[1146f02dc5] - crypto: return correct bit length in KeyObject's asymmetricKeyDetails (Filip Skokan) #46106
[961710bb72] - (SEMVER-MINOR) crypto: add KeyObject Symbol.toStringTag (Filip Skokan) #46043
[9cfdac6c82] - deps: V8: cherry-pick e39af94dd18e (Lu Yahan) #46142
[26cde8efb7] - deps: update simdutf to 3.1.0 (Node.js GitHub Bot) #46257
[3f9fb37130] - deps: cherrypick simdutf patch (Jiawen Geng) #46194
[4ff2822836] - deps: bump googletest to 2023.01.13 (Jiawen Geng) #46198
[49556247d2] - deps: add /deps/**/.github/ to .gitignore (Luigi Pinca) #46091
[0c4df83e0d] - deps: add simdutf version to metadata (Mike Roth) #46145
[69aafc3ddd] - deps: update simdutf to 2.1.0 (Node.js GitHub Bot) #46128
[a266daccb5] - deps: update corepack to 0.15.3 (Node.js GitHub Bot) #46037
[6cd70573eb] - deps: upgrade npm to 9.3.1 (npm team) #46242
[679aae2da8] - deps: upgrade npm to 9.3.0 (npm team) #46193
[38dd5061f2] - dgram: sync the old handle state to new handle (theanarkh) #46041
[e36af49b35] - doc: fix mismatched arguments of NodeEventTarget (Deokjin Kim) #45678
[58b836f7c4] - doc: update events API example to have runnable code (Deokjin Kim) #45760
[5c350298b4] - doc: add note to tls docs about secureContext availability (Tim Gerk) #46224
[90924ce198] - doc: add text around collaborative expectations (Michael Dawson) #46121
[2d328355d4] - doc: update to match changed --dns-result-order default (Mordy Tikotzky) #46148
[1015a606b7] - doc: add Node-API media link (Kevin Eady) #46189
[6e355efcff] - doc: update http.setMaxIdleHTTPParsers arguments (Debadree Chatterjee) #46168
[f18ab9405a] - doc: use "file system" instead of "filesystem" (Rich Trott) #46178
[1b45713b00] - doc: https update default request timeout (Marco Ippolito) #46184
[4c88721e2f] - doc: make options of readableStream.pipeTo as optional (Deokjin Kim) #46180
[538c53f010] - doc: add PerformanceObserver.supportedEntryTypes to doc (theanarkh) #45962
[eef7489d24] - doc: duplex and readable from uncaught execption warning (Marco Ippolito) #46135
[686fe585b5] - doc: remove outdated sections from maintaining-v8 (Antoine du Hamel) #46137
[2e826ad528] - doc: fix (EC)DHE remark in TLS docs (Tobias Nießen) #46114
[2e22b29add] - doc: fix ERR_TLS_RENEGOTIATION_DISABLED text (Tobias Nießen) #46122
[e222a2f1d1] - doc: fix spelling in SECURITY.md (Vaishno Chaitanya) #46124
[7718e82f0d] - doc: abort controller emits error in child process (Debadree Chatterjee) #46072
[76408bc1ed] - doc: fix event.cancelBubble documentation (Deokjin Kim) #45986
[82023f2570] - doc: update output of example in inspector (Deokjin Kim) #46073
[a42fc512b6] - doc: add personal pronouns option (Filip Skokan) #46118
[fafae5955d] - doc: mention how to run ncu-ci citgm (Rafael Gonzaga) #46090
[e1fd2f24d9] - doc: include updating release optional step (Rafael Gonzaga) #46089
[1996e610fd] - doc: describe argument of Symbol.for (Deokjin Kim) #46019
[b002330216] - doc,crypto: fix WebCryptoAPI import keyData and export return (Filip Skokan) #46076
[fa3e0c86c7] - esm: mark importAssertions as required (Antoine du Hamel) #46164
[f85a8e4c59] - events: add initEvent to Event (Deokjin Kim) #46069
[5bdfaae680] - events: change status of event.returnvalue to legacy (Deokjin Kim) #46175
[ad7846fe97] - events: change status of event.cancelBubble to legacy (Deokjin Kim) #46146
[5304c89682] - events: change status of event.srcElement to legacy (Deokjin Kim) #46085
[3dcdab3f16] - events: check signal before listener (Deokjin Kim) #46054
[907d67de76] - http: refactor to use validateHeaderName (Deokjin Kim) #46143
[ae5141cb8a] - http: writeHead if statusmessage is undefined dont override headers (Marco Ippolito) #46173
[6e7f9fbc1d] - http: refactor to use min of validateNumber for maxTotalSockets (Deokjin Kim) #46115
[069a30bc4e] - (SEMVER-MINOR) http: join authorization headers (Marco Ippolito) #45982
[68cde4cbcc] - lib: add webstreams to Duplex.from() (Debadree Chatterjee) #46190
[4d73ea708b] - lib: use kEmptyObject and update JSDoc in webstreams (Deokjin Kim) #46183
[1cfa2e6762] - lib: refactor to use validate function (Deokjin Kim) #46101
[2eb87f23c9] - lib: reuse invalid state errors on webstreams (Rafael Gonzaga) #46086
[8684dae8d9] - lib: fix incorrect use of console intrinsic (Colin Ihrig) #46044
[d044ed1d3e] - meta: update AUTHORS (Node.js GitHub Bot) #46215
[5261560757] - meta: update AUTHORS (Node.js GitHub Bot) #46130
[1b557bbee8] - meta: update comment in CODEOWNERS to better reflect current policy (Antoine du Hamel) #45944
[54896ab011] - module: fix unintended mutation (Antoine du Hamel) #46108
[bd98e5baba] - node-api: disambiguate napi_add_finalizer (Chengzhong Wu) #45401
[f0508894d6] - perf_hooks: fix checking range of options.figures in createHistogram (Deokjin Kim) #45999
[e482d5e42d] - src: fix endianness of simdutf (Yagiz Nizipli) #46257
[e2c47cdfad] - src: make BuiltinLoader threadsafe and non-global (Anna Henningsen) #45942
[36ae3ccff3] - src: replace unreachable code with static_assert (Tobias Nießen) #46209
[9d55a1f9a1] - src: hide kMaxDigestMultiplier outside HKDF impl (Tobias Nießen) #46206
[d3d62ed82c] - src: distinguish env stopping flags (Chengzhong Wu) #45907
[e85f76686c] - src: remove return after abort (Shelley Vohr) #46172
[7dc9a53b18] - src: remove unnecessary semicolons (Shelley Vohr) #46171
[28af831d5a] - src: use simdutf for converting externalized builtins to UTF-16 (Anna Henningsen) #46119
[e8eaa490af] - src: use constant strings for memory info names (Chengzhong Wu) #46087
[f4559a1354] - src: fix typo in node_snapshotable.cc (Vadim) #46103
[ca8ff08a5c] - src: keep PipeWrap::Open function consistent with TCPWrap (theanarkh) #46064
[a936eaeb34] - src: speed up process.getActiveResourcesInfo() (Darshan Sen) #46014
[5cf595659f] - src,lib: the handle keeps loop alive in cluster rr mode (theanarkh) #46161
[18695595e1] - stream: fix pipeline calling end on destination more than once (Debadree Chatterjee) #46226
[e5f53b51f0] - stream: implement finished() for ReadableStream and WritableStream (Debadree Chatterjee) #46205
[2f23f17f93] - test: reduce fs-write-optional-params flakiness (LiviaMedeiros) #46238
[255f177108] - test: enable more case of bad buffer in fs.write (Deokjin Kim) #46236
[c09b2036c7] - test: update postject to 1.0.0-alpha.4 (Node.js GitHub Bot) #46212
[4ac5c7180f] - test: refactor to avoid mutation of global by a loader (Michaël Zasso) #46220
[bbf9da8e2c] - test: improve test coverage for WHATWG TextDecoder (Juan José) #45241
[4f491d368c] - test: add fix so that test exits if port 42 is unprivileged (Suyash Nayan) #45904
[6e2f7228f3] - test: use os.availableParallelism() (Deokjin Kim) #46003
[c77b0da512] - test: fix flaky test-runner-exit-code.js (Colin Ihrig) #46138
[f309e2acb6] - test: update Web Events WPT (Deokjin Kim) #46051
[0f60bc9bbc] - test: add test to once() in event lib (Jonathan Diaz) #46126
[8a8b18678a] - test,esm: validate more edge cases for dynamic imports (Antoine du Hamel) #46059
[4d3743938f] - test_runner: update comment to comply with eslint no-fallthrough rule (Antoine du Hamel) #46258
[653b108fdc] - tools: update eslint to 8.32.0 (Node.js GitHub Bot) #46258
[a4b0c916e0] - tools: update lint-md-dependencies (Node.js GitHub Bot) #46214
[f4465e656d] - tools: fix macro name in update-undici (Almeida) #46217
[1aa4534c6f] - tools: add automation for updating postject dependency (Darshan Sen) #46157
[c150b312cd] - tools: update create-or-update-pull-request-action (Michaël Zasso) #46169
[c68a043400] - tools: update eslint to 8.31.0 (Node.js GitHub Bot) #46131
[ac90e419d1] - tools: update lint-md-dependencies to [email protected] (Node.js GitHub Bot) #46129
[750fcf84eb] - tools: move update-eslint.sh to dep_updaters/ (Luigi Pinca) #46088
[2e8750a18c] - tools: make update-eslint.sh work with npm@9 (Luigi Pinca) #46088
[e90a3a6eff] - tools: fix lint rule recommendation (Colin Ihrig) #46044
[0985ef8bfb] - tools: add ArrayPrototypeConcat to the list of primordials to avoid (Antoine du Hamel) #44445
[ed69a3af92] - tools: add prefer-proto rule (Jordan Harband) #46083
[4c1c20fae2] - trace_events: refactor to use validateStringArray (Deokjin Kim) #46012
[6c8a81d2dc] - vm: refactor to use validate function (Deokjin Kim) #46176

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

bricss and marco-ippolito reacted with thumbs up emoji

bricss and marco-ippolito reacted with hooray emoji

bricss, debadree25, and marco-ippolito reacted with heart emoji

bricss, marco-ippolito, and debadree25 reacted with rocket emoji

All reactions

👍 2 reactions
🎉 2 reactions
❤️ 3 reactions
🚀 3 reactions

npm-cli-bot and others added 30 commits

January 18, 2023 10:14


          deps: upgrade npm to 9.3.0

679aae2

PR-URL: #46193
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Tobias Nießen <[email protected]>
Reviewed-By: Jiawen Geng <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Myles Borins <[email protected]>
Reviewed-By: Rich Trott <[email protected]>


          deps: upgrade npm to 9.3.1

6cd7057

PR-URL: #46242
Reviewed-By: Ruy Adorno <[email protected]>
Reviewed-By: Myles Borins <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>


          tools: add prefer-proto rule

ed69a3a

fixup: add support for `Object.create(null)`

fixup: extend to any 1-argument Object.create call

fixup: add tests
PR-URL: #46083
Backport-PR-URL: #46239
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Geoffrey Booth <[email protected]>
Reviewed-By: Yagiz Nizipli <[email protected]>
Reviewed-By: Mohammed Keyvanzadeh <[email protected]>
Reviewed-By: Darshan Sen <[email protected]>
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Jacob Smith <[email protected]>
Reviewed-By: Antoine du Hamel <[email protected]>


          http: join authorization headers

069a30b

PR-URL: #45982
Backport-PR-URL: #46240
Fixes: #45699
Reviewed-By: Matteo Collina <[email protected]>
Reviewed-By: Yagiz Nizipli <[email protected]>
Reviewed-By: Paolo Insogna <[email protected]>


          tools: add ArrayPrototypeConcat to the list of primordials to avoid

0985ef8

PR-URL: #44445
Reviewed-By: Matteo Collina <[email protected]>
Reviewed-By: Joyee Cheung <[email protected]>
Reviewed-By: Rafael Gonzaga <[email protected]>
Reviewed-By: Tobias Nießen <[email protected]>


          deps: update corepack to 0.15.3

a266dac

PR-URL: #46037
Reviewed-By: Mohammed Keyvanzadeh <[email protected]>
Reviewed-By: Moshe Atlow <[email protected]>
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Juan José Arboleda <[email protected]>


          dgram: sync the old handle state to new handle

38dd506

PR-URL: #46041
Reviewed-By: Matteo Collina <[email protected]>
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: Paolo Insogna <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Darshan Sen <[email protected]>
Reviewed-By: Juan José Arboleda <[email protected]>


          src: speed up process.getActiveResourcesInfo()

a936eae

This change reduces the number of calls that were crossing the JS-C++
boundary to 1 and also removes the need for calling Array::New()
multiple times internally and ArrayPrototypeConcat-ing the results
later on, thus improving performance.

Refs: #44445 (review)
Signed-off-by: Darshan Sen <[email protected]>
PR-URL: #46014
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Chengzhong Wu <[email protected]>


          crypto: add KeyObject Symbol.toStringTag

961710b

PR-URL: #46043
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Yagiz Nizipli <[email protected]>
Reviewed-By: Tobias Nießen <[email protected]>


          tools: fix lint rule recommendation

e90a3a6

The lint rule recommends destructuring console out of the
internal/console/global.js export. However, that does not
exist. The top level export is actually the console object.

PR-URL: #46044
Reviewed-By: Yagiz Nizipli <[email protected]>
Reviewed-By: Filip Skokan <[email protected]>
Reviewed-By: Tobias Nießen <[email protected]>
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Darshan Sen <[email protected]>
Reviewed-By: Juan José Arboleda <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>


          lib: fix incorrect use of console intrinsic

8684dae

The console object was not being frozen because the intrinsic
freezing code was accessing undefined instead of the console
object.

PR-URL: #46044
Reviewed-By: Yagiz Nizipli <[email protected]>
Reviewed-By: Filip Skokan <[email protected]>
Reviewed-By: Tobias Nießen <[email protected]>
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Darshan Sen <[email protected]>
Reviewed-By: Juan José Arboleda <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>


          events: check signal before listener

3dcdab3

In WPT Events, TypeError is expected if both listener and
signal are null. But checking listener doesn't throw TypeError.
So check signal before listener because checking signal throws
TypeError if signal is null.

PR-URL: #46054
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Juan José Arboleda <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Minwoo Jung <[email protected]>


          node-api: disambiguate napi_add_finalizer

bd98e5b

napi_add_finalizer doesn't support any operations related to
napi_wrap. Remove the ambiguous statements in the doc about
napi_wrap and avoid reusing the v8impl::Wrap call.

PR-URL: #45401
Reviewed-By: Michael Dawson <[email protected]>


          test,esm: validate more edge cases for dynamic imports

8a8b186

PR-URL: #46059
Reviewed-By: Juan José Arboleda <[email protected]>
Reviewed-By: Jacob Smith <[email protected]>
Reviewed-By: Geoffrey Booth <[email protected]>
Reviewed-By: Minwoo Jung <[email protected]>


          doc: describe argument of Symbol.for

1996e61

Symbol is described with argument, but argument of Symbol.for
is ommited. So describe argument of Symbol.for for consistency.

Refs: https://github.com/nodejs/node/blob/main/doc/contributing/using-symbols.md#symbolstring
PR-URL: #46019
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Chengzhong Wu <[email protected]>


          meta: update comment in CODEOWNERS to better reflect current policy

1b557bb

We do not enforce signing off by any code owners when landing PRs.

PR-URL: #45944
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Darshan Sen <[email protected]>
Reviewed-By: Joyee Cheung <[email protected]>
Reviewed-By: Geoffrey Booth <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Chengzhong Wu <[email protected]>
Reviewed-By: Michael Dawson <[email protected]>
Reviewed-By: Beth Griggs <[email protected]>
Reviewed-By: Danielle Adams <[email protected]>


          tools: make update-eslint.sh work with npm@9

2e8750a

Make the `update-eslint.sh` script work with `npm@9`.

PR-URL: #46088
Refs: https://github.com/nodejs/node/actions/runs/3814364920/jobs/6488613583#step:3:64
Reviewed-By: Ruy Adorno <[email protected]>
Reviewed-By: Antoine du Hamel <[email protected]>


          tools: move update-eslint.sh to dep_updaters/

750fcf8

Move the `update-eslint.sh` script to the `dep_updaters/` directory.

PR-URL: #46088
Refs: https://github.com/nodejs/node/actions/runs/3814364920/jobs/6488613583#step:3:64
Reviewed-By: Ruy Adorno <[email protected]>
Reviewed-By: Antoine du Hamel <[email protected]>


          doc: include updating release optional step

e1fd2f2

PR-URL: #46089
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Ruy Adorno <[email protected]>
Reviewed-By: Beth Griggs <[email protected]>


          doc: mention how to run ncu-ci citgm

fafae59

PR-URL: #46090
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Yagiz Nizipli <[email protected]>
Reviewed-By: Beth Griggs <[email protected]>


          doc: add personal pronouns option

a42fc51

PR-URL: #46118
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>


          src: keep PipeWrap::Open function consistent with TCPWrap

ca8ff08

PR-URL: #46064
Reviewed-By: Darshan Sen <[email protected]>
Reviewed-By: Anna Henningsen <[email protected]>


          doc: update output of example in inspector

82023f2

Actual output of promise example in inspector is mismatched.

PR-URL: #46073
Reviewed-By: Kohei Ueno <[email protected]>
Reviewed-By: Darshan Sen <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>


          doc: fix event.cancelBubble documentation

76408bc

Setter of `event.cancelBubble` is a no-op if the value passed to it is
not `true`.

Refs: https://dom.spec.whatwg.org/#interface-event
PR-URL: #45986
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Anna Henningsen <[email protected]>


          doc: abort controller emits error in child process

7718e82

Fixes: #46036
PR-URL: #46072
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Anna Henningsen <[email protected]>


          doc,crypto: fix WebCryptoAPI import keyData and export return

b002330

PR-URL: #46076
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Tobias Nießen <[email protected]>


          src: fix typo in node_snapshotable.cc

f4559a1

ImmeidateInfo -> ImmediateInfo

PR-URL: #46103
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: Chengzhong Wu <[email protected]>
Reviewed-By: Darshan Sen <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>


          events: change status of event.srcElement to legacy

5304c89

`event.srcElement` is described as legacy in spec.

Refs: https://dom.spec.whatwg.org/#interface-event
PR-URL: #46085
Reviewed-By: Darshan Sen <[email protected]>
Reviewed-By: Benjamin Gruenbaum <[email protected]>
Reviewed-By: Antoine du Hamel <[email protected]>


          crypto: return correct bit length in KeyObject's asymmetricKeyDetails

1146f02

PR-URL: #46106
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: Ben Noordhuis <[email protected]>


          module: fix unintended mutation

54896ab

Refs: #46061 (comment)
PR-URL: #46108
Reviewed-By: Rafael Gonzaga <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Geoffrey Booth <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Jacob Smith <[email protected]>

RafaelGSS force-pushed the v19.5.0-proposal branch from ecb598d to 17145d8 Compare

January 20, 2023 18:39

RafaelGSS added the request-ci Add this label to start a Jenkins CI on a PR. Only starts once the PR has an approving review. label

Jan 20, 2023

nodejs-github-bot commented Jan 20, 2023

Copy link

Copy Markdown

Collaborator

V8 CI: https://ci.nodejs.org/job/node-test-commit-v8-linux/nodes=benchmark-ubuntu1804-intel-64,v8test=v8test/5131/

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

nodejs-github-bot commented Jan 20, 2023

Copy link

Copy Markdown

Collaborator

V8 CI: https://ci.nodejs.org/job/node-test-commit-v8-linux/nodes=rhel8-s390x,v8test=v8test/5131/

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

nodejs-github-bot commented Jan 20, 2023

Copy link

Copy Markdown

Collaborator

V8 CI: https://ci.nodejs.org/job/node-test-commit-v8-linux/nodes=rhel8-ppc64le,v8test=v8test/5131/

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. Only starts once the PR has an approving review. label

Jan 20, 2023

This comment was marked as outdated.


          2023-01-24, Version 19.5.0 (Current)

07c9241

Notable changes:

* http:
  * (SEMVER-MINOR) join authorization headers (Marco Ippolito) [#45982](#45982)
* lib:
  * add webstreams to Duplex.from() (Debadree Chatterjee) [#46190](#46190)
* stream:
  * implement finished() for ReadableStream and WritableStream (Debadree Chatterjee) [#46205](#46205)

PR-URL: #46286
Signed-off-by: RafaelGSS <[email protected]>

RafaelGSS force-pushed the v19.5.0-proposal branch from 17145d8 to 07c9241 Compare

January 20, 2023 18:48

This comment was marked as outdated.

github-actions Bot mentioned this pull request

Jan 21, 2023

CI Reliability 2023-01-21 nodejs/reliability#490

Open

14 tasks

nodejs-github-bot commented Jan 21, 2023 •

edited by RafaelGSS

Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link

Copy Markdown

Collaborator

CI: https://ci.nodejs.org/job/node-test-pull-request/49113/ ✔️
V8: https://ci.nodejs.org/job/node-test-commit-v8-linux/5131/ ✔️
CITGM: https://ci.nodejs.org/job/citgm-smoker/3077/
CITGM(v19.x): https://ci.nodejs.org/job/citgm-smoker/3081/
📦 https://ci-release.nodejs.org/job/iojs+release/9098/ -> https://ci-release.nodejs.org/job/iojs+release/9099/

CITGM Results ✔️

FAILURE: 15 failures in 3077 not present in 3081


┌────────────────────────┬───────────────────────┬───────────────────────┬───────────────────┬────────────────┬───────────────┬───────────────┐
│        (index)         │           0           │           1           │         2         │       3        │       4       │       5       │
├────────────────────────┼───────────────────────┼───────────────────────┼───────────────────┼────────────────┼───────────────┼───────────────┤
│       rhel8-x64        │                       │                       │                   │                │               │               │
│       win-vs2019       │                       │                       │                   │                │               │               │
│     rhel8-ppc64le      │    'async-v3.2.4'     │      'gulp-util'      │ 'inherits-v2.0.4' │ 'level-v8.0.0' │ 'pump-v3.0.0' │ 'tape-v5.6.3' │
│        osx1015         │ 'tough-cookie-v4.1.2' │                       │                   │                │               │               │
│     ubuntu1804-64      │    'async-v3.2.4'     │                       │                   │                │               │               │
│      rhel8-s390x       │                       │                       │                   │                │               │               │
│      aix72-ppc64       │    'async-v3.2.4'     │ 'tough-cookie-v4.1.2' │                   │                │               │               │
│      debian10-x64      │    'bcrypt-v5.1.0'    │   'fastify-v4.11.0'   │   'pino-v8.8.0'   │ 'pump-v3.0.0'  │               │               │
│ fedora-last-latest-x64 │ 'tough-cookie-v4.1.2' │                       │                   │                │               │               │
└────────────────────────┴───────────────────────┴───────────────────────┴───────────────────┴────────────────┴───────────────┴───────────────┘

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

github-actions Bot mentioned this pull request

Jan 22, 2023

CI Reliability 2023-01-22 nodejs/reliability#491

Open

13 tasks

jaydenseric commented Jan 22, 2023

Copy link

Copy Markdown

Contributor

I can't see mention of the test runner reporters in this proposal, will they be included in the v19.5.0 release? Per previous v19.4.0 discussion: #46061 (comment) .

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

bricss commented Jan 22, 2023

Copy link

Copy Markdown

Contributor

@jaydenseric check dis out #46092 (comment) 👀

RafaelGSS reacted with heart emoji

jaydenseric reacted with eyes emoji

All reactions

❤️ 1 reaction
👀 1 reaction

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

github-actions Bot mentioned this pull request

Jan 23, 2023

CI Reliability 2023-01-23 nodejs/reliability#492

Open

13 tasks

debadree25 mentioned this pull request

Jan 23, 2023

doc: include webstreams in finished() and Duplex.from() parameters #46312

Merged

github-actions Bot mentioned this pull request

Jan 24, 2023

CI Reliability 2023-01-24 nodejs/reliability#493

Open

13 tasks

richardlau approved these changes

Jan 24, 2023

View reviewed changes

ruyadorno approved these changes

Jan 24, 2023

View reviewed changes

ruyadorno commented Jan 24, 2023

Copy link

Copy Markdown

Member

super small and optional but if for some reason you decide to amend the release commit, it would be nice to remove the markdown format from the PRs linked in the commit message.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

RafaelGSS added a commit that referenced this pull request

Jan 24, 2023


          Working on v19.5.1

e0fddab

PR-URL: #46286

RafaelGSS merged commit 07c9241 into v19.x

Jan 24, 2023

RafaelGSS deleted the v19.5.0-proposal branch

January 24, 2023 21:14

RafaelGSS added a commit that referenced this pull request

Jan 24, 2023


          2023-01-24, Version 19.5.0 (Current)

40a206c

Notable changes:

* http:
  * (SEMVER-MINOR) join authorization headers (Marco Ippolito) [#45982](#45982)
* lib:
  * add webstreams to Duplex.from() (Debadree Chatterjee) [#46190](#46190)
* stream:
  * implement finished() for ReadableStream and WritableStream (Debadree Chatterjee) [#46205](#46205)

PR-URL: #46286
Signed-off-by: RafaelGSS <[email protected]>

RafaelGSS added a commit to RafaelGSS/nodejs.org that referenced this pull request

Jan 24, 2023


          Blog: v19.5.0 release post

567a8e6

Refs: nodejs/node#46286

RafaelGSS mentioned this pull request

Jan 24, 2023

Blog: v19.5.0 release post nodejs/nodejs.org#5032

Merged

RafaelGSS added a commit to nodejs/nodejs.org that referenced this pull request

Jan 24, 2023


          Blog: v19.5.0 release post (#5032)

712a266

Refs: nodejs/node#46286

github-actions Bot mentioned this pull request

Jan 25, 2023

CI Reliability 2023-01-25 nodejs/reliability#494

Open

12 tasks

Reviewers

ruyadorno

ruyadorno approved these changes

richardlau

richardlau approved these changes

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

release

Issues and PRs related to Node.js releases.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

20 participants

Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.

You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.

GeezerVison

Conversation

RafaelGSS commented Jan 20, 2023 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

2023-01-24, Version 19.5.0 (Current), @RafaelGSS

Notable Changes

Commits

Uh oh!

nodejs-github-bot commented Jan 20, 2023

Uh oh!

nodejs-github-bot commented Jan 20, 2023

Uh oh!

nodejs-github-bot commented Jan 20, 2023

Uh oh!

This comment was marked as outdated.

This comment was marked as outdated.

nodejs-github-bot commented Jan 21, 2023 • edited by RafaelGSS Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

jaydenseric commented Jan 22, 2023

Uh oh!

bricss commented Jan 22, 2023

Uh oh!

ruyadorno commented Jan 24, 2023

Uh oh!

Reviewers

Uh oh!

Assignees

Labels

Projects

Milestone

Development

Uh oh!

20 participants

RafaelGSS commented Jan 20, 2023 •

edited

Loading

Uh oh!

There was an error while loading. Please reload this page.

nodejs-github-bot commented Jan 21, 2023 •

edited by RafaelGSS

Loading

Uh oh!

There was an error while loading. Please reload this page.