- Oct 27, 2020
-
-
Michaël Zasso authored
This adds support for nullish coalescing, optional chaining and numeric separators. The acorn-numeric-separator plugin can be removed. PR-URL: https://github.com/nodejs/node/pull/35791 Reviewed-By:
Colin Ihrig <cjihrig@gmail.com> Reviewed-By:
Rich Trott <rtrott@gmail.com> Reviewed-By:
Daijiro Wachi <daijiro.wachi@gmail.com> Reviewed-By:
Matteo Collina <matteo.collina@gmail.com> Reviewed-By:
James M Snell <jasnell@gmail.com> Reviewed-By:
Shelley Vohr <codebytere@gmail.com> Reviewed-By:
Jiawen Geng <technicalcute@gmail.com>
-
raisinten authored
PR-URL: https://github.com/nodejs/node/pull/35807 Reviewed-By:
Richard Lau <rlau@redhat.com> Reviewed-By:
Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By:
Gireesh Punathil <gpunathi@in.ibm.com> Reviewed-By:
Juan José Arboleda <soyjuanarbol@gmail.com> Reviewed-By:
Luigi Pinca <luigipinca@gmail.com>
-
- Oct 26, 2020
-
-
Daniel Bevenius authored
This reverts Commit 3ff2aece ("src: ignore GCC -Wcast-function-type for v8.h") and Commit 2462a2c5 ("src: fix ignore GCC -Wcast-function-type for older compilers") as this has now been included in the V8 version being used. PR-URL: https://github.com/nodejs/node/pull/35768 Refs: https://github.com/v8/v8/commit/f08cbfdc4051266e4200b6e26775d35307b1259b Reviewed-By:
Michaël Zasso <targos@protonmail.com> Reviewed-By:
Minwoo Jung <nodecorelab@gmail.com> Reviewed-By:
David Carlier <devnexen@gmail.com>
-
Daniel Bevenius authored
PR-URL: https://github.com/nodejs/node/pull/35716 Reviewed-By:
-
Daniel Bevenius authored
This reverts commit f21cf456 as this has now been included in the V8 version being used. PR-URL: https://github.com/nodejs/node/pull/35758 Refs: https://github.com/v8/v8/commit/f08cbfdc4051266e4200b6e26775d35307b1259b Reviewed-By:
Denys Otrishko <shishugi@gmail.com> Reviewed-By:
-
Pooja D.P authored
PR-URL: https://github.com/nodejs/node/pull/35554 Reviewed-By:
Anna Henningsen <anna@addaleax.net> Reviewed-By:
Harshitha K P <harshitha014@gmail.com> Reviewed-By:
-
Pooja D.P authored
PR-URL: https://github.com/nodejs/node/pull/35517 Reviewed-By:
-
bcoe authored
Turns off coverage comments for the time being, until we can sort out issues. PR-URL: https://github.com/nodejs/node/pull/35800 Reviewed-By:
Stephen Belanger <admin@stephenbelanger.com> Reviewed-By:
-
Rich Trott authored
The kbd element has already been added in the keybindings table, but we missed it in one line in the prose. PR-URL: https://github.com/nodejs/node/pull/35737 Reviewed-By:
Roman Reiss <me@silverwind.io> Reviewed-By:
Franziska Hinkelmann <franziska.hinkelmann@gmail.com>
-
Shubham Parihar authored
PR-URL: https://github.com/nodejs/node/pull/35786 Reviewed-By:
Ricky Zhou <0x19951125@gmail.com>
-
- Oct 25, 2020
-
-
bcoe authored
Cleanup logic in Makefile for coverage. Update BUILDING.md accordingly. PR-URL: https://github.com/nodejs/node/pull/35767 Reviewed-By:
Joyee Cheung <joyeec9h3@gmail.com>
-
ax1 authored
Indentation for #fs_fs_promises_api is level 2 but it should be level 3. PR-URL: https://github.com/nodejs/node/pull/35771 Reviewed-By:
-
Rich Trott authored
Remove note indicating that the APIs can change at any time. This module migrated from Experimental to Stable 6 years ago (in io.js 1.4.2). PR-URL: https://github.com/nodejs/node/pull/35774 Reviewed-By:
-
Vse Mozhe Buty authored
PR-URL: https://github.com/nodejs/node/pull/35764 Reviewed-By:
-
Yash Ladha authored
Using the variable name in the comment and justifying the type seems redundant to me and instead it should defined the entity which it is acting, like in our case it is acting as a flag to control the flow in streams. PR-URL: https://github.com/nodejs/node/pull/35728 Reviewed-By:
Robert Nagy <ronagy@icloud.com> Reviewed-By:
Anto Aravinth <anto.aravinth.cse@gmail.com>
-
Turner Jabbour authored
PR-URL: https://github.com/nodejs/node/pull/35437 Reviewed-By:
Anatoli Papirovski <apapirovski@mac.com> Reviewed-By:
-
Pooja D.P authored
example code added to the process.setgroups() API doc property in process.md PR-URL: https://github.com/nodejs/node/pull/35738 Reviewed-By:
-
Momtchil Momtchev authored
We reinject when the sockets has already waiting data, remarked by @mildsunrise Co-authored-by:
Alba Mendez <me@alba.sh> PR-URL: https://github.com/nodejs/node/pull/35678 Reviewed-By:
-
Momtchil Momtchev authored
Reinject the data already received from the TLS socket when the HTTP2 client is attached with a delay Fixes: https://github.com/nodejs/node/issues/35475 PR-URL: https://github.com/nodejs/node/pull/35678 Reviewed-By:
-
Vincent Boivin authored
Allow the user to update the server settings after using http2.createSecureServer() or after using http2.createServer(). Fixes: https://github.com/nodejs/node/issues/35353 PR-URL: https://github.com/nodejs/node/pull/35383 Reviewed-By:
Yongsheng Zhang <zyszys98@gmail.com> Reviewed-By:
-
ExE Boss authored
Fixes: https://github.com/nodejs/node/issues/35740 Refs: https://github.com/nodejs/node/pull/31553 Refs: https://github.com/nodejs/node/pull/32953 Refs: https://github.com/nodejs/node/pull/33991 Refs: https://github.com/nodejs/node/pull/34001 Refs: https://github.com/nodejs/node/pull/34055 Refs: https://github.com/nodejs/node/pull/34962 Co-authored-by:
Trivikram Kamat <trivikr.dev@gmail.com> Co-authored-by:
Masashi Hirano <shisama07@gmail.com> Reviewed-By:
Pranshu Srivastava <rexagod@gmail.com> Reviewed-By:
-
Josh Dague authored
Fixes: https://github.com/nodejs/node/issues/34818 PR-URL: https://github.com/nodejs/node/pull/34824 Reviewed-By:
-
- Oct 24, 2020
-
-
Momtchil Momtchev authored
The debug sprintf doesn't support %.* specifiers Fixes: https://github.com/nodejs/node/issues/35688 PR-URL: https://github.com/nodejs/node/pull/35694 Reviewed-By:
-
Ian Sutherland authored
PR-URL: https://github.com/nodejs/node/pull/35683 Reviewed-By:
Ben Coe <bencoe@gmail.com> Reviewed-By:
-
Lee, Bonggi authored
Signed-off-by:
Lee, Bonggi <iyabong@gmail.com> PR-URL: https://github.com/nodejs/node/pull/34964 Reviewed-By:
-
Antoine du Hamel authored
Refs: https://github.com/nodejs/node/pull/35525 PR-URL: https://github.com/nodejs/node/pull/35782 Reviewed-By:
-
Rich Trott authored
* Make the 7 instances of breakOnSigint text blocks consistent. * Use present tense. * Use kbd element for keystrokes. * Minor style edits. PR-URL: https://github.com/nodejs/node/pull/35692 Reviewed-By:
-
Pooja D.P authored
PR-URL: https://github.com/nodejs/node/pull/35515 Reviewed-By:
Christian Clauss <cclauss@me.com> Reviewed-By:
-
Yash Ladha authored
Fix duplicate logic in stream destroy as the same logic is being shared across methods and thus can be encapsulated into a single method. PR-URL: https://github.com/nodejs/node/pull/35727 Reviewed-By:
Benjamin Gruenbaum <benjamingr@gmail.com> Reviewed-By:
-
Guy Bedford authored
PR-URL: https://github.com/nodejs/node/pull/35745 Reviewed-By:
Jan Krems <jan.krems@gmail.com> Reviewed-By:
Myles Borins <myles.borins@gmail.com> Reviewed-By:
Ujjwal Sharma <ryzokuken@disroot.org> Reviewed-By:
Michael Dawson <midawson@redhat.com>
-
Myles Borins authored
Only imports has the experimental status right now but technically all ESM related features are currently experimental. The list also doesn't appear to be in any specific order so I've grouped the experimental features together. PR-URL: https://github.com/nodejs/node/pull/35741 Reviewed-By:
Guy Bedford <guybedford@gmail.com> Reviewed-By:
Derek Lewis <DerekNonGeneric@inf.is> Reviewed-By:
-
- Oct 23, 2020
-
-
Rich Trott authored
Cherry-pick 12c8b4d1 Original commit message: This commit is a suggestion for adding a rule for NULL usages in the code base. This will currently report a number of errors which could be ignored using // NOLINT (readability/null_usage) PR-URL: https://github.com/nodejs/node/pull/17373 Reviewed-By:
Jon Moss <me@jonathanmoss.me> Reviewed-By:
Timothy Gu <timothygu99@gmail.com> Reviewed-By:
Michael Dawson <michael_dawson@ca.ibm.com> Reviewed-By:
Sakthipriyan Vairamani <thechargingvolcano@gmail.com> Reviewed-By:
Tobias Nießen <tniessen@tnie.de> Refs: https://github.com/nodejs/node/commit/12c8b4d15471cb6211b39c3a2ca5b10fa4b9f12b Cherry-pick fc81e801 Original commit message: Update cpplint.py to check for inline headers when the corresponding header is already included. PR-URL: https://github.com/nodejs/node/pull/21521 Reviewed-By:
Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By:
Ruben Bridgewater <ruben@bridgewater.de> Reviewed-By:
Daniel Bevenius <daniel.bevenius@gmail.com> Refs: https://github.com/nodejs/node/commit/0a25ace9c35b62ece4d32fd90b326d8063265109 Cherry-pick afa9a720 Original commit message: tools: refloat update link to google styleguide for cpplint This commit updates two old links to Google's C++ styleguide which currently result in a 404 when accessed. PR-URL: https://github.com/nodejs/node/pull/30876 Reviewed-By:
Richard Lau <riclau@uk.ibm.com> Reviewed-By:
Gus Caplan <me@gus.host> Reviewed-By:
-
Rich Trott authored
https://github.com/cpplint/cpplint/releases/tag/1.5.0 PR-URL: https://github.com/nodejs/node/pull/35719 Reviewed-By:
-
Rich Trott authored
PR-URL: https://github.com/nodejs/node/pull/35719 Reviewed-By:
-
- Oct 22, 2020
-
-
Antoine du Hamel authored
PR-URL: https://github.com/nodejs/node/pull/35750 Reviewed-By:
-
Pooja D.P authored
PR-URL: https://github.com/nodejs/node/pull/35710 Reviewed-By:
-
Tierney Cyren authored
PR-URL: https://github.com/nodejs/node/pull/35712 Reviewed-By:
-
Joyee Cheung authored
Add a v8.stopCoverage() API to stop the coverage collection started by NODE_V8_COVERAGE - this would be useful in conjunction with v8.takeCoverage() if the user don't want to emit the coverage at the process exit but still want to collect it on demand at some point. PR-URL: https://github.com/nodejs/node/pull/33807 Reviewed-By:
-
Joyee Cheung authored
Add an v8.takeCoverage() API that allows the user to write the coverage started by NODE_V8_COVERAGE to disk on demand. The coverage can be written multiple times during the lifetime of the process, each time the execution counter will be reset. When the process is about to exit, one last coverage will still be written to disk. Also refactors the internal profiler connection code so that we use the inspector response id to identify the profile response instead of using an ad-hoc flag in C++. PR-URL: https://github.com/nodejs/node/pull/33807 Reviewed-By:
-
Benjamin Coe authored
Collect Windows and C++ coverage. Configure codecov so that comments are more concise and are only left when coverage varies. PR-URL: https://github.com/nodejs/node/pull/35670 Fixes: https://github.com/nodejs/node/issues/35696 Refs: https://github.com/nodejs/node/pull/35653 Refs: https://github.com/nodejs/node/issues/35646 Reviewed-By:
-
