- Apr 30, 2017
-
-
Refael Ackermann authored
PR-URL: https://github.com/nodejs/node/pull/12731 Reviewed-By:
Anna Henningsen <anna@addaleax.net> Reviewed-By:
Evan Lucas <evanlucas@me.com> Reviewed-By:
James M Snell <jasnell@gmail.com> Reviewed-By:
Gibson Fahnestock <gibfahn@gmail.com> Reviewed-By:
Colin Ihrig <cjihrig@gmail.com> Reviewed-By:
Jeremiah Senkpiel <fishrock123@rocketmail.com> Reviewed-By:
Daijiro Wachi <daijiro.wachi@gmail.com> Reviewed-By:
Timothy Gu <timothygu99@gmail.com>
-
Brian White authored
PR-URL: https://github.com/nodejs/node/pull/12456 Reviewed-By:
Benjamin Gruenbaum <benjamingr@gmail.com> Reviewed-By:
-
Brian White authored
PR-URL: https://github.com/nodejs/node/pull/12456 Reviewed-By:
-
Brian White authored
PR-URL: https://github.com/nodejs/node/pull/12456 Reviewed-By:
-
Brian White authored
PR-URL: https://github.com/nodejs/node/pull/12456 Reviewed-By:
-
Brian White authored
PR-URL: https://github.com/nodejs/node/pull/12456 Reviewed-By:
-
Brian White authored
PR-URL: https://github.com/nodejs/node/pull/12456 Reviewed-By:
-
Brian White authored
PR-URL: https://github.com/nodejs/node/pull/12456 Reviewed-By:
-
Brian White authored
PR-URL: https://github.com/nodejs/node/pull/12456 Reviewed-By:
-
Brian White authored
PR-URL: https://github.com/nodejs/node/pull/12456 Reviewed-By:
-
Brian White authored
PR-URL: https://github.com/nodejs/node/pull/12456 Reviewed-By:
-
Brian White authored
PR-URL: https://github.com/nodejs/node/pull/12456 Reviewed-By:
-
Brian White authored
PR-URL: https://github.com/nodejs/node/pull/12456 Reviewed-By:
-
Brian White authored
PR-URL: https://github.com/nodejs/node/pull/12456 Reviewed-By:
-
gwer authored
Start the transition to Array.prototype.includes() and String.prototype.includes(). This commit refactors most of the comparisons of Array.prototype.indexOf() and String.prototype.indexOf() return values with -1 to the former methods in tests. PR-URL: https://github.com/nodejs/node/pull/12604 Refs: https://github.com/nodejs/node/issues/12586 Reviewed-By:
Alexey Orlenko <eaglexrlnk@gmail.com> Reviewed-By:
-
dave-k authored
Ensure that cluster interoperates with the --inspect-brk option. This does not test for --debug-brk. Fixes: https://github.com/nodejs/node/issues/11420 PR-URL: https://github.com/nodejs/node/pull/12503 Reviewed-By:
-
Cameron Little authored
The unhandled promise rejection warning uses a template literal and prints the reason a promise was rejected. If rejecting with a symbol, the symbol failed to convert to a string and the process crashed. Now, symbols are casted to strings and the process does not crash. Fixes: https://github.com/nodejs/node/issues/11637 PR-URL: https://github.com/nodejs/node/pull/11640 Reviewed-By:
-
Cameron Little authored
This allows the common.checkWarning() test method to accept a map of warning names to description(s), to allow testing code that generates multiple types of warnings. PR-URL: https://github.com/nodejs/node/pull/11640 Reviewed-By:
-
Alexey Orlenko authored
* Add the personal pronoun for @aqrln. * Fix incorrectly ordered @lucamaraschi entry. PR-URL: https://github.com/nodejs/node/pull/12750 Reviewed-By:
-
Zuzana Svetlikova authored
Porting https://github.com/nodejs/node/pull/12392 to master Ref: https://github.com/nodejs/node/pull/12392 Fixes: https://github.com/nodejs/node/issues/10388 PR-URL: https://github.com/nodejs/node/pull/12676 Reviewed-By:
Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By:
-
James M Snell authored
PR-URL: https://github.com/nodejs/node/pull/12717 Reviewed-By:
-
James M Snell authored
PR-URL: https://github.com/nodejs/node/pull/12717 Reviewed-By:
-
- Apr 29, 2017
-
-
alohaglenn authored
PR-URL: https://github.com/nodejs/node/pull/12599 Reviewed-By:
-
Anna Henningsen authored
Add documentation for `common.crashOnUnhandledRejection()`. Ref: https://github.com/nodejs/node/pull/12489/files/a9c2078a60bc3012dc6156df19772697a56a2517#r113737423 PR-URL: https://github.com/nodejs/node/pull/12699 Reviewed-By:
Rich Trott <rtrott@gmail.com> Reviewed-By:
Santiago Gimeno <santiago.gimeno@gmail.com> Reviewed-By:
Luigi Pinca <luigipinca@gmail.com> Reviewed-By:
Yuta Hiroto <hello@about-hiroppy.com> Reviewed-By:
Michael Dawson <michael_dawson@ca.ibm.com> Reviewed-By:
-
Michaël Zasso authored
PR-URL: https://github.com/nodejs/node/pull/12621 Reviewed-By:
Refael Ackermann <refack@gmail.com> Reviewed-By:
-
Anna Henningsen authored
The test fixtures create multiple node::Environments that all use the uv_default_loop(), and since the test does not clean up the handles created by Environment::Start(), the default libuv loop structure contains dangling pointers after the first Environment is freed, which then means that creating new handles leads to memory corruption. PR-URL: https://github.com/nodejs/node/pull/12621 Reviewed-By:
Michaël Zasso <targos@protonmail.com>
-
Daniel Bevenius authored
Currently when configure --without-ssl the test will throw the following error: bad option: --use-openssl-ca This commit checks if crypto was enabled and skips the crypto related tests if that is the case. PR-URL: https://github.com/nodejs/node/pull/12692 Reviewed-By:
Sam Roberts <vieuxtech@gmail.com>/>
-
Rich Trott authored
test-benchmark-child-process failures reveal that child-process-exec-stdout benchmark sometimes leaves around a stray yes.exe process. Add code to terminate the process. PR-URL: https://github.com/nodejs/node/pull/12658 Ref: https://github.com/nodejs/node/issues/12560 Reviewed-By:
Bartosz Sosnowski <bartosz@janeasystems.com> Reviewed-By:
-
Vse Mozhet Byt authored
Delete unused method call. PR-URL: https://github.com/nodejs/node/pull/12685 Refs: https://github.com/nodejs/node/pull/6171 Reviewed-By:
-
Vse Mozhet Byt authored
* Update an example according to an actual REPL session. * Replace an arrow function with a common function to hold `this`. PR-URL: https://github.com/nodejs/node/pull/12684 Reviewed-By:
-
coreybeaumont authored
Found that libuv had a path character limit of 108. Used path.relative() to set prefix with relative path. Also added comments explaining the use of a relative path. PR-URL: https://github.com/nodejs/node/pull/12601 Reviewed-By:
-
James M Snell authored
Use the more efficient module.exports = {} pattern, restructure imports from bindings, requires. PR-URL: https://github.com/nodejs/node/pull/12681 Reviewed-By:Vse Mozhet Byt <vsemozhetbyt@gmail.com>
-
James M Snell authored
PR-URL: https://github.com/nodejs/node/pull/12681 Reviewed-By:
-
Michael Dawson authored
Add the initial documentation for the N-API This PR is a result of work in the abi-stable-node repo: https://github.com/nodejs/abi-stable-node/tree/doc, with this PR being the cumulative work on the documentation sections in that repo with the following contributors in alphabetical order: Author: Arunesh Chandra <arunesh.chandra@microsoft.com> Author: Gabriel Schulhof <gabriel.schulhof@intel.com> Author: Hitesh Kanwathirtha <hiteshk@microsoft.com> Author: Jason Ginchereau <jasongin@microsoft.com> Author: Michael Dawson <michael_dawson@ca.ibm.com> Author: Sampson Gao <sampsong@ca.ibm.com> Author: Taylor Woll <taylor.woll@microsoft.com> PR-URL: https://github.com/nodejs/node/pull/12549 Reviewed-By:
-
Timothy Gu authored
PR-URL: https://github.com/nodejs/node/pull/12683 Reviewed-By:
Joyee Cheung <joyeec9h3@gmail.com>
-
Timothy Gu authored
PR-URL: https://github.com/nodejs/node/pull/12683 Reviewed-By:
-
Michael Dawson authored
Add specifics as to the level of hardware supported for power platforms. Power 8 is the first processor level to support ppc little endian. Set the minimum level for AIX to Power7 for Node version 8 and later. This will allow the potential to leverage new instructions and optimizations going forward. We have spoken to the AIX team and they agree this makes sense. Earlier processor levels will continue to be supported for version 4.x and 6.x on AIX. PR-URL: https://github.com/nodejs/node/pull/12679 Reviewed-By:
Richard Lau <riclau@uk.ibm.com>
-
cjihrig authored
PR-URL: https://github.com/nodejs/node/pull/12669 Reviewed-By:
-
cjihrig authored
This commit refactors test-whatwg-url-tojson.js to remove ESLint comments. PR-URL: https://github.com/nodejs/node/pull/12669 Reviewed-By:
-
The fs watch test was not run on AIX till now because of the known issue, https://github.com/nodejs/node/issues/5085. Now that it is completed, this guard can be removed. PR-URL: https://github.com/nodejs/node/pull/12687 Reviewed-By:
Brian White <mscdex@mscdex.net>
-
