Commits · 0028241fb4c19d04066b006bca4f3e0b3b403d90 · github.com / v8-riscv / node

Nov 24, 2017

doc: update http URLs to https in README.md · 0028241f

Ronald Eddy Jr authored Nov 22, 2017

PR-URL: https://github.com/nodejs/node/pull/17264


Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Myles Borins <myles.borins@gmail.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Anatoli Papirovski <apapirovski@mac.com>

0028241f

doc: update http URLs to https in doc/api · 7ab78400

Ronald Eddy Jr authored Nov 22, 2017

PR-URL: https://github.com/nodejs/node/pull/17263


Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Myles Borins <myles.borins@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Anatoli Papirovski <apapirovski@mac.com>

7ab78400

doc: update http URLs to https in GOVERNANCE.md · 7f446f0f

Ronald Eddy Jr authored Nov 22, 2017

PR-URL: https://github.com/nodejs/node/pull/17262


Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Myles Borins <myles.borins@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Anatoli Papirovski <apapirovski@mac.com>

7f446f0f

doc: update http URLs to https in CONTRIBUTING.md · de3bb8f8

Ronald Eddy Jr authored Nov 22, 2017

PR-URL: https://github.com/nodejs/node/pull/17261


Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Myles Borins <myles.borins@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Anatoli Papirovski <apapirovski@mac.com>

de3bb8f8

Nov 23, 2017

Revert "src, tools: add debug symbols for node internals" · 088bba31

Ben Noordhuis authored Nov 23, 2017

This reverts commit 446c1ecf.

Reverted for breaking `make test-v8`.  The "find all relevant system
headers" logic in `tools/gen-postmortem-metadata.py` needs more work.

Also reverts commit e46c3f74.
("tools: fix typo in gen-postmortem-metadata.py".)

PR-URL: https://github.com/nodejs/node/pull/17272
Refs: https://github.com/nodejs/node/pull/14901


Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Daniel Bevenius <daniel.bevenius@gmail.com>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
Reviewed-By: Matheus Marchini <matheusdot@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>

088bba31

doc: add SharedArrayBuffer to Buffer documentation · bd0e36db

Thomas den Hollander authored Sep 20, 2017

PR-URL: https://github.com/nodejs/node/pull/15489


Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Michaël Zasso <targos@protonmail.com>

bd0e36db

internal: add emitExperimentalWarning function · b08c7321

Cody Deckard authored Oct 25, 2017

Adds a new emitExperimentalWarning function to internal/util to warn
users that they are using an experimental feature and that feature can
change at any time.

PR-URL: https://github.com/nodejs/node/pull/16497
Refs: https://github.com/nodejs/node/issues/9036


Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Michaël Zasso <targos@protonmail.com>

b08c7321

doc: document resolve hook formats · e5d4aeb6

Lucas Azzola authored Oct 22, 2017

Add `"dynamic"` to the list of supported `format`s returned by a
custom resolve hook.

Add a table describing the meaning of each `format`.

PR-URL: https://github.com/nodejs/node/pull/16375
Refs: https://github.com/nodejs/node/pull/15445


Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com>
Reviewed-By: Yuta Hiroto <hello@about-hiroppy.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Michaël Zasso <targos@protonmail.com>

e5d4aeb6

tools: fix typo in gen-postmortem-metadata.py · e46c3f74

Daniel Bevenius authored Nov 23, 2017

PR-URL: https://github.com/nodejs/node/pull/17268


Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>

e46c3f74

test: use common.crashOnUnhandledRejection · 317f596b

Jason Chung authored Nov 22, 2017

PR-URL: https://github.com/nodejs/node/pull/17233


Reviewed-By: Myles Borins <myles.borins@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>

317f596b

tools/doc: add tools/remark-* to eslintignore · c2c4dd4c

Ivan Wei authored Nov 22, 2017

PR-URL: https://github.com/nodejs/node/pull/17240


Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>

c2c4dd4c

test: use common.crashOnUnhandledRejection() · 6b874077

sorarize@gmail.com authored Nov 22, 2017

Add common.crashOnUnhandledRejection to
test/parallel/test-microtask-queue-run.js

PR-URL: https://github.com/nodejs/node/pull/17232


Reviewed-By: Myles Borins <myles.borins@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>

6b874077

test: use common.crashOnUnhandledRejection · 01dc6d26

Kurt Hsu authored Nov 22, 2017

PR-URL: https://github.com/nodejs/node/pull/17229


Reviewed-By: James M Snell <jasnell@gmail.com>

01dc6d26

test: add common.crashOnHandleRejection · 6b56be21

jackyen authored Nov 22, 2017

PR-URL: https://github.com/nodejs/node/pull/17225


Reviewed-By: James M Snell <jasnell@gmail.com>

6b56be21

test: add crashonUnhandledRejection · 3f19ecda

danielLin authored Nov 22, 2017

PR-URL: https://github.com/nodejs/node/pull/17237


Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: James M Snell <jasnell@gmail.com>

3f19ecda

benchmark,path: remove unused variables · fbb9eef4

薛定谔的猫 authored Oct 06, 2017

PR-URL: https://github.com/nodejs/node/pull/15789


Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>

fbb9eef4

process: Send signal name to signal handlers · 9b2cf1c8

Robert Rossmann authored Sep 25, 2017

PR-URL: https://github.com/nodejs/node/pull/15606


Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Evan Lucas <evanlucas@me.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Bartosz Sosnowski <bartosz@janeasystems.com>

9b2cf1c8

process: improve unhandled rejection message · 13db29b2

Madara Uchiha authored Nov 20, 2017

PR-URL: https://github.com/nodejs/node/pull/17158
Refs: https://github.com/nodejs/node/pull/16768


Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Evan Lucas <evanlucas@me.com>
Reviewed-By: James M Snell <jasnell@gmail.com>

13db29b2

test: keep coverage reports after coverage-clean · c6b7052b

Anatoli Papirovski authored Sep 19, 2017

Add coverage folder to .gitignore and remove it from the list
of files & folders delete by coverage-clean.

PR-URL: https://github.com/nodejs/node/pull/15470


Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Evan Lucas <evanlucas@me.com>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>

c6b7052b

test: add test on unhandled rejection · 18cf0960

Larry Lu authored Nov 22, 2017

PR-URL: https://github.com/nodejs/node/pull/17228


Reviewed-By: Myles Borins <myles.borins@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>

18cf0960

test: use common.crashOnUnhandledRejection · e34ccebd

aryung chen authored Nov 22, 2017

PR-URL: https://github.com/nodejs/node/pull/17221


Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>

e34ccebd

test: use common.crashOnUnhandledRejection · ab99a890

Zack Yang authored Nov 22, 2017

Deal with crash case for unhandled rejection.

PR-URL: https://github.com/nodejs/node/pull/17217


Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: James M Snell <jasnell@gmail.com>

ab99a890

test: add common.crashOnUnhandledRejection() · f7c8d235

Scya597 authored Nov 22, 2017

PR-URL: https://github.com/nodejs/node/pull/17212


Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>

f7c8d235

doc: fs.readFile is async but not partitioned · a59ad82a

Jamie Davis authored Nov 20, 2017

This change was suggested during the discussion of #17054.

PR-URL: https://github.com/nodejs/node/pull/17154


Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com>

a59ad82a

test,doc: document where common modules go · 17db46c4

Gibson Fahnestock authored Oct 08, 2017

Keep the `require('../common')` separate from other common modules, as
it's the only line that must be there.

PR-URL: https://github.com/nodejs/node/pull/16089


Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: James M Snell <jasnell@gmail.com>

17db46c4

src: add public API for managing NodePlatform · 27281124

Cheng Zhao authored Nov 13, 2017

PR-URL: https://github.com/nodejs/node/pull/16981


Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>

27281124

assert: use Object.is comparison in .strictEqual · 493340f5

Ruben Bridgewater authored Nov 13, 2017

This aligns assert.strictEqual and assert.notStrictEqual with
assert.deepStrictEqual to use the Object.is() comparison instead
of strict equality.

PR-URL: https://github.com/nodejs/node/pull/17003


Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
Reviewed-By: Timothy Gu <timothygu99@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>

493340f5

doc: add description for inspector-only console methods. · 982c6741

Benjamin Zaslavsky authored Nov 14, 2017

Description inspired by dev tools reference and inspector err messages

Added:
* intro
* console.debug()
* console.dirxml()
* console.markTimeline()
* console.profile()
* console.profileEnd()
* console.table()
* console.timeStamp()
* console.timeline()
* console.timelineEnd()

PR-URL: https://github.com/nodejs/node/pull/17004
Fixes: https://github.com/nodejs/node/issues/16755


Reviewed-By: Khaidi Chu <i@2333.moe>
Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>

982c6741

zlib: remove unnecessary else branch · b98027ee

john authored Nov 15, 2017

PR-URL: https://github.com/nodejs/node/pull/17057


Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Franziska Hinkelmann <franziska.hinkelmann@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Evan Lucas <evanlucas@me.com>
Reviewed-By: Anatoli Papirovski <apapirovski@mac.com>
Reviewed-By: James M Snell <jasnell@gmail.com>

b98027ee

test: remove unlink function which is needless · b28a28a5

buji authored Nov 20, 2017

PR-URL: https://github.com/nodejs/node/pull/17119


Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Daniel Bevenius <daniel.bevenius@gmail.com>

b28a28a5

test: dont need to remove nonexistent directory · fdbee334

buji authored Nov 19, 2017

the TempDirection is empty , so tempdir/test1 and tempdir/test2
are nonexistent. we dont need to delete nonexistent directories.

PR-URL: https://github.com/nodejs/node/pull/17119


Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Daniel Bevenius <daniel.bevenius@gmail.com>

fdbee334

fs: use arrow functions instead of `.bind` and `self` · a68b8b70

Weijia Wang authored Nov 20, 2017

PR-URL: https://github.com/nodejs/node/pull/17137


Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>

a68b8b70

net: remove deprecated getters for internals · 3701b023

Anna Henningsen authored Nov 19, 2017

Remove the getters introduced in 75a19fb3.

PR-URL: https://github.com/nodejs/node/pull/17141
Refs: https://github.com/nodejs/node/pull/14449


Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: Anatoli Papirovski <apapirovski@mac.com>
Reviewed-By: Timothy Gu <timothygu99@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Minwoo Jung <minwoo@nodesource.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Evan Lucas <evanlucas@me.com>
Reviewed-By: James M Snell <jasnell@gmail.com>

3701b023

doc: use better terminology for build machines · 8f504077

Anna Henningsen authored Nov 19, 2017

PR-URL: https://github.com/nodejs/node/pull/17142


Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com>
Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com>
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Anatoli Papirovski <apapirovski@mac.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>

8f504077

stream: improve the error message of `ERR_INVALID_ARG_TYPE` · 0a948b90

Weijia Wang authored Nov 20, 2017

The `expected` argument of `ERR_INVALID_ARG_TYPE` can be an
array, which is better than a single string.

PR-URL: https://github.com/nodejs/node/pull/17145


Reviewed-By: Anatoli Papirovski <apapirovski@mac.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>

0a948b90

doc: use "JavaScript" instead of "Javascript" · 8f2c366b

Rich Trott authored Nov 18, 2017

PR-URL: https://github.com/nodejs/node/pull/17163


Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Alexey Orlenko <eaglexrlnk@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>

8f2c366b

doc: prepare for v8/V8 linting in doc text · 911f1632

Rich Trott authored Nov 18, 2017

PR-URL: https://github.com/nodejs/node/pull/17163


Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Alexey Orlenko <eaglexrlnk@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>

911f1632

doc: add capitalization styling to STYLE_GUIDE · e577d7db

Rich Trott authored Nov 18, 2017

This is mostly an excuse to remind people that it's "V8" and not "v8"
when referring to the JavaScript engine.

PR-URL: https://github.com/nodejs/node/pull/17163


Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Alexey Orlenko <eaglexrlnk@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>

e577d7db

test: use common.crashOnUnhandledRejection() · a3a987b4

Ivan Wei authored Nov 22, 2017

add common.crashOnUnhandledRejectio to
test-http2-window-size.js

PR-URL: https://github.com/nodejs/node/pull/17227


Reviewed-By: Myles Borins <myles.borins@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: James M Snell <jasnell@gmail.com>

a3a987b4

test: add common.crashOnUnhandledRejection() · 96093981

Kyle Yu authored Nov 22, 2017

add common.crashOnUnhandledRejection() in test-wasm-simple.js

PR-URL: https://github.com/nodejs/node/pull/17236


Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Myles Borins <myles.borins@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>

96093981