summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* doc: change STYLE-GUIDE to STYLE_GUIDEDean Coakley2017-02-221-0/+0
| | | | | | | | | | | PR-URL: https://github.com/nodejs/node/pull/11460 Fixes: https://github.com/nodejs/node/issues/11456 Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Claudio Rodriguez <cjrodr@yahoo.com> Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com> Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
* lib: remove unnecessary assignments with _extendSakthipriyan Vairamani (thefourtheye)2017-02-224-8/+8
| | | | | | | | | | | | | | The first parameter to `util._extend` is the target object. Assigning the target object to the result of `util._extend` is not necessary. PR-URL: https://github.com/nodejs/node/pull/11364 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Yuta Hiroto <hello@about-hiroppy.com> Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: Brian White <mscdex@mscdex.net>
* 2017-02-21, Version 7.6.0 (Current)Italo A. Casas2017-02-212-1/+145
| | | | | | | | | | | | | | | | | | Notable changes: * deps: * update V8 to 5.5 (Michaël Zasso) [#11029](https://github.com/nodejs/node/pull/11029) * upgrade libuv to 1.11.0 (cjihrig) [#11094](https://github.com/nodejs/node/pull/11094) * add node-inspect 1.10.4 (Jan Krems) [#10187](https://github.com/nodejs/node/pull/10187) * upgrade zlib to 1.2.11 (Sam Roberts) [#10980](https://github.com/nodejs/node/pull/10980) * lib: build `node inspect` into `node` (Anna Henningsen) [#10187](https://github.com/nodejs/node/pull/10187) * crypto: Remove expired certs from CNNIC whitelist (Shigeki Ohtsu) [#9469](https://github.com/nodejs/node/pull/9469) * inspector: add --inspect-brk (Josh Gavant) [#11149](https://github.com/nodejs/node/pull/11149) * fs: allow WHATWG URL objects as paths (James M Snell) [#10739](https://github.com/nodejs/node/pull/10739) * src: support UTF-8 in compiled-in JS source files (Ben Noordhuis) [#11129](https://github.com/nodejs/node/pull/11129) * url: extend url.format to support WHATWG URL (James M Snell) [#10857](https://github.com/nodejs/node/pull/10857) PR-URL: https://github.com/nodejs/node/pull/11185
* 2017-02-21, Version 6.10.0 'Boron' (LTS)Myles Borins2017-02-212-1/+195
| | | | | | | | | | | | | | | | | | | | | | | | Notable Changes: The SEMVER-MINOR changes include: * crypto: allow adding extra certs to well-known CAs (Sam Roberts) https://github.com/nodejs/node/pull/9139 * deps: Upgrade INTL ICU to version 58 (Steven R. Loomis) https://github.com/nodejs/node/pull/9234 * process: add `process.memoryUsage.external` (Fedor Indutny) https://github.com/nodejs/node/pull/9587 * src: add wrapper for process.emitWarning() (Sam Roberts) https://github.com/nodejs/node/pull/9139 Notable SEMVER-PATCH changes include: * fs: cache non-symlinks in realpathSync. (Jeremy Yallop) https://github.com/nodejs/node/pull/10253 * repl: allow autocompletion for scoped packages (Evan Lucas) https://github.com/nodejs/node/pull/10296 PR-URL: https://github.com/nodejs/node/pull/10974
* 2017-02-21, Version 4.8.0 'Argon' (LTS)Myles Borins2017-02-212-1/+149
| | | | | | | | | | | | | | | | | | | | | | | | Notable Changes: * child_process: add shell option to spawn() (cjihrig) https://github.com/nodejs/node/pull/4598 * crypto: * add ALPN Support (Shigeki Ohtsu) https://github.com/nodejs/node/pull/2564 * allow adding extra certs to well-known CAs (Sam Roberts) https://github.com/nodejs/node/pull/9139 * deps: * v8: expose statistics about heap spaces (Ben Ripkens) https://github.com/nodejs/node/pull/4463 * fs: add the fs.mkdtemp() function. (Florian MARGAINE) https://github.com/nodejs/node/pull/5333 * process: * add `externalMemory` to `process` (Fedor Indutny) https://github.com/nodejs/node/pull/9587 * add process.cpuUsage() (Patrick Mueller) https://github.com/nodejs/node/pull/10796 PR-URL: https://github.com/nodejs/node/pull/10973
* benchmark: add dgram bind(+/- params) benchmarkVse Mozhet Byt2017-02-211-0/+38
| | | | | | Refs: https://github.com/nodejs/node/pull/11242 PR-URL: https://github.com/nodejs/node/pull/11313 Reviewed-By: Brian White <mscdex@mscdex.net>
* console: fixup `console.dir()` error handlingAnna Henningsen2017-02-213-50/+57
| | | | | | | | | | | | Apply the `console: do not emit error events` changes properly to `console.dir()`. This was overlooked in f18e08d820dde161788d95a5603546ceca021e90 (https://github.com/nodejs/node/pull/9744). Ref: https://github.com/nodejs/node/commit/f18e08d820dde161788d95a5603546ceca021e90#commitcomment-20934407 PR-URL: https://github.com/nodejs/node/pull/11443 Reviewed-By: James M Snell <jasnell@gmail.com>
* tools: enable one-var-declaration-per-line ESLint ruleMichaël Zasso2017-02-2113-18/+29
| | | | | | | | | | | | This rule enforces new lines around variable declarations. It is configured to spot only variables that are initialized. PR-URL: https://github.com/nodejs/node/pull/11462 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Yuta Hiroto <hello@about-hiroppy.com> Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
* test: add regex check to test-module-loadingTarang Hirani2017-02-211-2/+3
| | | | | | | | | Also removes extraneous argument. PR-URL: https://github.com/nodejs/node/pull/11413 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com> Reviewed-By: Yuta Hiroto <hello@about-hiroppy.com>
* doc: restrict the ES.Next features usage in testsDavidCai2017-02-211-2/+6
| | | | | | | | PR-URL: https://github.com/nodejs/node/pull/11452 Refs: https://github.com/nodejs/node/pull/11290 Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
* test: remove redundant additional url testsJoyee Cheung2017-02-211-85/+4
| | | | | | | | | | | | url-tests-additional.js now is a placeholder since existing test cases have been upstreamed. This is now a temporary place for test cases not upstreamed yet. Refs: https://github.com/w3c/web-platform-tests/pull/4693 PR-URL: https://github.com/nodejs/node/pull/11439 Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: Timothy Gu <timothygu99@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
* test: synchronize WPT url test dataJoyee Cheung2017-02-212-41/+894
| | | | | | | | | | Synchronize url-tests to upstream b207902 and comment out incompatible tests PR-URL: https://github.com/nodejs/node/pull/11439 Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: Timothy Gu <timothygu99@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
* test: remove WHATWG URL test data file extensionJoyee Cheung2017-02-216-8/+14
| | | | | | | | | | Make url-tests a js file so we can comment out incompatible tests PR-URL: https://github.com/nodejs/node/pull/11439 Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: Timothy Gu <timothygu99@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
* http: add new functions to OutgoingMessageBrian White2017-02-203-1/+140
| | | | | | | | | | PR-URL: https://github.com/nodejs/node/pull/10805 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Matteo Collina <matteo.collina@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Roman Reiss <me@silverwind.io> Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com> Reviewed-By: Sam Roberts <vieuxtech@gmail.com>
* doc: add comment for net.Server's error eventQianJin20132017-02-201-2/+5
| | | | | | | | Fixes: https://github.com/nodejs/node/issues/9710 PR-URL: https://github.com/nodejs/node/pull/11136 Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Sam Roberts <vieuxtech@gmail.com>
* doc: fix sorting in API referencesVse Mozhet Byt2017-02-2012-472/+472
| | | | | | PR-URL: https://github.com/nodejs/node/pull/11331 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Sam Roberts <vieuxtech@gmail.com>
* deps: upgrade zlib to 1.2.11Sam Roberts2017-02-2082-1720/+3202
| | | | | | PR-URL: https://github.com/nodejs/node/pull/10980 Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: Shigeki Ohtsu <ohtsu@iij.ad.jp>
* tls: new tls.TLSSocket() supports sec ctx optionsSam Roberts2017-02-203-3/+18
| | | | | | | | | | | Add support to new tls.TLSSocket() to create a SecureContext object with all its supported options, in the same way they are supported for all the other APIs that need SecureContext objects. Fix: https://github.com/nodejs/node/issues/10538 PR-URL: https://github.com/nodejs/node/pull/11005 Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: James M Snell <jasnell@gmail.com>
* test: make tls-socket-default-options tests runSam Roberts2017-02-201-45/+49
| | | | | | | | | | | | | Because of a poorly constructed test, only one of the two test vectors ran. The test also failed to cover the authentication error that occurs when the server's certificate is not trusted. Both issues are fixed. Fix: https://github.com/nodejs/node/issues/10538 PR-URL: https://github.com/nodejs/node/pull/11005 Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: James M Snell <jasnell@gmail.com>
* doc: add version meta for SSL_CERT_DIR/FILESam Roberts2017-02-201-0/+6
| | | | | | | | | The metadata markup for when SSL_CERT_DIR and SSL_CERT_FILE added in 33012e9866cbb43db562dd6a3feffe8cd4ec5838 is missing. PR-URL: https://github.com/nodejs/node/pull/11007 Reviewed-By: Evan Lucas <evanlucas@me.com> Reviewed-By: James M Snell <jasnell@gmail.com>
* tls: use emitWarning() for dhparam < 2048 bitsJames M Snell2017-02-204-8/+6
| | | | | | | | | | | | | When a dhparam less than 2048 bits was used, a warning was being printed directly to console.error using an internalUtil.trace function that was not used anywhere else. This replaces it with a proper process warning and removes the internalUtil.trace function. PR-URL: https://github.com/nodejs/node/pull/11447 Reviewed-By: Shigeki Ohtsu <ohtsu@iij.ad.jp> Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: Rod Vagg <rod@vagg.org>
* util: cleanup internalUtil.deprecateJames M Snell2017-02-192-9/+3
| | | | | | | | | | There were two functions `deprecate` and `_deprecate` that were really just aliases of each other. Simplify PR-URL: https://github.com/nodejs/node/pull/11450 Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
* test: improve coverage in test-crypto.dhEric Christie2017-02-191-9/+26
| | | | | | | | PR-URL: https://github.com/nodejs/node/pull/11253 Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
* cluster: properly handle --inspect-{brk,port}Ali Ijaz Sheikh2017-02-192-1/+3
| | | | | | | | | | PR-URL: https://github.com/nodejs/node/pull/11386 Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: Josh Gavant <josh.gavant@outlook.com> Reviewed-By: James M Snell <jasnell@gmail.com>
* test: add error checking in callbackRich Trott2017-02-191-0/+1
| | | | | | | | | | | | Add assert.ifError() for test-dgram-send-callback-buffer-length. PR-URL: https://github.com/nodejs/node/pull/11446 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Santiago Gimeno <santiago.gimeno@gmail.com> Reviewed-By: Yuta Hiroto <hello@about-hiroppy.com> Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
* test: refactor test-http-response-splittingArseniy Maximov2017-02-191-9/+9
| | | | | | | | | | | | | | * move repeated code to function * remove unneeded `common.mustCall()` usage with function arguments that are not callbacks * add error message checking PR-URL: https://github.com/nodejs/node/pull/11429 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: Сковорода Никита Андреевич <chalkerx@gmail.com>
* test: test bottom-up merge sort in URLSearchParamsDaijiro Wachi2017-02-191-2/+15
| | | | | | | | | | | The bottom-up iterative stable merge sort is called only when the length of provided value is larger than 100. Added a test for it. PR-URL: https://github.com/nodejs/node/pull/11399 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com> Reviewed-By: Timothy Gu <timothygu99@gmail.com> Reviewed-By: Michaël Zasso <targos@protonmail.com>
* test: add test cases for pathYuta Hiroto2017-02-191-0/+2
| | | | | | | | | | | Add test cases for the following line. https://github.com/nodejs/node/blob/0f2f8ef/lib/path.js#L1364 https://github.com/nodejs/node/blob/0f2f8ef/lib/path.js#L1347 PR-URL: https://github.com/nodejs/node/pull/11453 Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
* test: add cases for unescape & unescapeBufferDaijiro Wachi2017-02-191-0/+23
| | | | | | | | | | | | | These two functions in the querystring are used as a fallback. To test them, two test cases were added which make errors that will be caught. PR-URL: https://github.com/nodejs/node/pull/11326 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Timothy Gu <timothygu99@gmail.com> Reviewed-By: Yuta Hiroto <hello@about-hiroppy.com> Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
* test: enhance test-common.jsRich Trott2017-02-191-1/+12
| | | | | | | | | | | | | | * refactor test-common.js so that global leak detection does not need to be disabled * add test for common.fail() PR-URL: https://github.com/nodejs/node/pull/11433 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Santiago Gimeno <santiago.gimeno@gmail.com> Reviewed-By: Michaël Zasso <targos@protonmail.com>
* util, debugger: remove internalUtil.errorJames M Snell2017-02-193-29/+10
| | | | | | | | | The internalUtil.error() function was only used by _debugger.js. PR-URL: https://github.com/nodejs/node/pull/11448 Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
* util: eliminate unnecessary exportsJames M Snell2017-02-193-25/+23
| | | | | | | | | | The getHiddenValue and setHiddenValue functions are exported from internalUtil for no really good reason PR-URL: https://github.com/nodejs/node/pull/11451 Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
* test: fix over-dependence on native promise implAli Ijaz Sheikh2017-02-191-2/+2
| | | | | | | | | | | | | In later versions of V8, the stack trace will be different than our current expectations for Promise methods. PR-URL: https://github.com/nodejs/node/pull/11437 Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Evan Lucas <evanlucas@me.com> Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com> Reviewed-By: Michaël Zasso <targos@protonmail.com>
* src: remove usage of deprecated debug APIYang Guo2017-02-191-1/+2
| | | | | | | | | | | | | Debug::GetMirror is used in contextify to force loading of the debug context. There is a better way to achieve this. PR-URL: https://github.com/nodejs/node/pull/11437 Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Evan Lucas <evanlucas@me.com> Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com> Reviewed-By: Michaël Zasso <targos@protonmail.com>
* child_process: refactor internal/child_process.jsArseniy Maximov2017-02-181-19/+27
| | | | | | | | | | | | * Prefer === to == where possible * Remove condition that will always be false * Prefer for-loop statements to forEach where possible for perfomance reasons PR-URL: https://github.com/nodejs/node/pull/11366 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Sam Roberts <vieuxtech@gmail.com> Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: Brian White <mscdex@mscdex.net>
* test: use expectsError in test-debug-agent.jsArseniy Maximov2017-02-181-5/+6
| | | | | | | | | | Use common.expectsError() in place of inline validation function in test-debug-agent.js. PR-URL: https://github.com/nodejs/node/pull/11410 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Michaël Zasso <targos@protonmail.com>
* test: add coverage for utf8CheckIncomplete()xiaoyu2017-02-181-0/+3
| | | | | | | | | | | This commit adds code coverage to utf8CheckIncomplete(), when the lastNeed is greater than buffer length. PR-URL: https://github.com/nodejs/node/pull/11419 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com> Reviewed-By: Michaël Zasso <targos@protonmail.com>
* child_process: remove empty if conditioncjihrig2017-02-181-3/+1
| | | | | | | | | | | This commit replaces an empty if-else-if with a single if condition. PR-URL: https://github.com/nodejs/node/pull/11427 Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: Santiago Gimeno <santiago.gimeno@gmail.com>
* test: add test for URLSearchParams inspectionDaijiro Wachi2017-02-181-0/+2
| | | | | | | | | | | The member methods of URLSearchParams should split across multiple lines with `util.inspect` depending on an option `breakLength`. PR-URL: https://github.com/nodejs/node/pull/11428 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Timothy Gu <timothygu99@gmail.com> Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com> Reviewed-By: Michaël Zasso <targos@protonmail.com>
* test: use expectsError in require-invalid-packageRich Trott2017-02-171-4/+4
| | | | | | | | | Use common.expectsError() in place of inline validation function in test-require-invalid-package. PR-URL: https://github.com/nodejs/node/pull/11409 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
* test: use common.expectsError()Rich Trott2017-02-171-5/+2
| | | | | | | | | | | | Use common.expectsError() instead of rolling own validation function in test-http-request-invalid-method-error.js. common.expectsError() didn't exist when the test was written, but now it does. PR-URL: https://github.com/nodejs/node/pull/11408 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com> Reviewed-By: Santiago Gimeno <santiago.gimeno@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
* test: refactor common.expectsError()Rich Trott2017-02-172-7/+9
| | | | | | | | | | | * Report values in assertions. * Strict equality match if message is a string. * instanceof/typeof instead of deprecated util.isRegExp() PR-URL: https://github.com/nodejs/node/pull/11381 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
* dgram: improve signature of Socket.prototype.sendChristopher Hiller2017-02-176-6/+134
| | | | | | | | | | | | | | | | | - Do not require presence of `address` parameter to use `callback` parameter; `address` is *always* optional - Improve exception messaging if `address` is invalid type - If `address` is an invalid type, guarantee a synchronously thrown exception - Update documentation to reflect signature changes - Add coverage around valid, undocumented types for `address` parameter. - Add coverage around known invalid, but uncovered, types for `address` parameter. PR-URL: https://github.com/nodejs/node/pull/10473 Reviewed-By: Sam Roberts <vieuxtech@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
* zlib: use RangeError/TypeError consistentlyJames M Snell2017-02-173-28/+30
| | | | | | | PR-URL: https://github.com/nodejs/node/pull/11391 Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Sam Roberts <vieuxtech@gmail.com>
* zlib: refactor zlib moduleJames M Snell2017-02-171-462/+403
| | | | | | | PR-URL: https://github.com/nodejs/node/pull/11391 Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Sam Roberts <vieuxtech@gmail.com>
* util: add createClassWrapper to internal/utilJames M Snell2017-02-172-0/+51
| | | | | | | | | | Utility function for wrapping an ES6 class with a constructor function that does not require the new keyword PR-URL: https://github.com/nodejs/node/pull/11391 Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Sam Roberts <vieuxtech@gmail.com>
* vm: refactor vm moduleJames M Snell2017-02-178-54/+66
| | | | | | | | Switch to the more efficient module.exports = {} pattern. PR-URL: https://github.com/nodejs/node/pull/11392 Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
* test: remove unused args and comparison fixAlexander2017-02-171-3/+3
| | | | | | | | | | | | Remove unused arguments and change non-strict comparison to the strict one in test. PR-URL: https://github.com/nodejs/node/pull/11396 Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: Yuta Hiroto <hello@about-hiroppy.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
* doc: improve test/README.mdJoyee Cheung2017-02-171-137/+144
| | | | | | | | | | | * Add description about the test directory * Add link to the test writing guide and the contributing guide * Use table to describe the directory layout and CI info PR-URL: https://github.com/nodejs/node/pull/11237 Fixes: https://github.com/nodejs/node/issues/11190 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Andreas Madsen <amwebdk@gmail.com>
* doc: add benchmark/README.md and fix guideJoyee Cheung2017-02-172-22/+278
| | | | | | | | | | | | * Write a new benchmark/README.md describing the benchmark directory layout and common API. * Fix the moved benchmarking guide accordingly, add tips about how to get the help text from the benchmarking tools. PR-URL: https://github.com/nodejs/node/pull/11237 Fixes: https://github.com/nodejs/node/issues/11190 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Andreas Madsen <amwebdk@gmail.com>
View Lorry Controller Status