Ecosyste.ms: OpenCollective
An open API service for software projects hosted on Open Collective.
github.com/electron/node
Node fork to make it suitable for embedding in Electron
https://github.com/electron/node
PR-URL: https://github.com/nodejs/node/pull/18845
Reviewed-By: Luigi Pinca <[email protected]...
Refs: https://github.com/nodejs/node/issues/11135
PR-URL: https://github.com/nodejs/node/pull/1...
e83adf87f58c7144e745ed18a79202577739b97c authored almost 7 years ago by dustinnewman98 <[email protected]>
PR-URL: https://github.com/nodejs/node/pull/18873
Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmai...
PR-URL: https://github.com/nodejs/node/pull/18872
Reviewed-By: Colin Ihrig <[email protected]>
R...
When using a debug build (on Windows specifically) the error case for
tls_wrap causes an assert ...
There are currently 3 places in Timers where the exact same code
appears. Instead create a helpe...
Currently the cctest target depend on the node_core_target_name
target. But it is the node_lib_t...
PR-URL: https://github.com/nodejs/node/pull/18780
Reviewed-By: James M Snell <[email protected]>...
PR-URL: https://github.com/nodejs/node/pull/18780
Reviewed-By: James M Snell <[email protected]>...
PR-URL: https://github.com/nodejs/node/pull/18780
Reviewed-By: James M Snell <[email protected]>...
Currenlty when configuring --debug-http2
/test/parallel/test-http2-getpackedsettings.js will seg...
When the async uv_fs_* call errors out synchronously in AsyncDestCall,
the after callbacks (e.g....
PR-URL: https://github.com/nodejs/node/pull/18739
Reviewed-By: Colin Ihrig <[email protected]>
R...
1. Adds fixer method
2. Extends test
PR-URL: https://github.com/nodejs/node/pull/16646
Refs: ht...
1. Fixer for crypto-check.js
2. Extends tests
PR-URL: https://github.com/nodejs/node/pull/16647...
f242d4b250e50105fc2455813159ee790ac5a198 authored almost 7 years ago by Shobhit Chittora <[email protected]>This brings the behaviour in line with the documentation.
PR-URL: https://github.com/nodejs/nod...
7ca4ca8cc36f7bf293f0d6717068eabb289262a6 authored almost 7 years ago by Sam Roberts <[email protected]>Use arrow functions and common.mustCall() and add a description.
PR-URL: https://github.com/nod...
c9b42c4c13a7b769dbb152b039cc28fe36072b75 authored almost 7 years ago by juggernaut451 <[email protected]>
Like man(7), mdoc(7) is a macro package for marking up computer manuals.
The main difference is ...
Adds note that accessing the fd of the IPC channel in any other way
than process.send, or using ...
- Use `common.mustCall()` to ensure that callbacks are called.
- Remove no longer needed variabl...
PR-URL: https://github.com/nodejs/node/pull/18827
Refs: https://github.com/v8/v8/compare/6.4.388...
PR-URL: https://github.com/nodejs/node/pull/18829
Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmai...
Add two regression tests for stdio over pipes.
test-stdio-pipe-access tests if accessing stdio ...
1bda746c1bf8f875cebb3a8948a57bbc75189240 authored almost 7 years ago by Bartosz Sosnowski <[email protected]>
It will also remove useless "code" variables by inlining
path.charCodeAt.
PR-URL: https://githu...
b404aa56c0e776c9722017ca020f9d04c7aa6de2 authored almost 7 years ago by Sergey Golovin <[email protected]>
`writable` is already set by the streams side, and
there is a handler waiting for the writable s...
PR-URL: https://github.com/nodejs/node/pull/18623
Refs: https://github.com/nodejs/node/pull/1821...
`node --prof foo.js` may not print the full profile log file, leaving
the last line broken (for ...
PR-URL: https://github.com/nodejs/node/pull/18777
Reviewed-By: Benjamin Gruenbaum <benjamingr@gm...
PR-URL: https://github.com/nodejs/node/pull/18777
Reviewed-By: Benjamin Gruenbaum <benjamingr@gm...
Signed-off-by: Yihong Wang <[email protected]>
PR-URL: https://github.com/nodejs/node/pull/18824
...
PR-URL: https://github.com/nodejs/node/pull/18803
Reviewed-By: Benjamin Gruenbaum <benjamingr@gm...
We've added a number of tests that hook into ESLint which can error
when running the test suite ...
The legacy MakeCallback deprecation was resulting in compile time
warnings in adddon tests. Fix ...
node_file was casting back and forth between v8::Resolver and
v8::Promise. This is unnecessary; ...
Upcoming changes to move away from synchronous I/O on the main
thread will imply that using the ...
PR-URL: https://github.com/nodejs/node/pull/18804
Reviewed-By: Weijia Wang <[email protected]>
R...
PR-URL: https://github.com/nodejs/node/pull/18823
Reviewed-By: Michael Dawson <michael_dawson@ca...
This enables the `no-multiple-empty-lines` eslint rule for the docs.
PR-URL: https://github.com...
69f7ce9cea1cc9b250365e6291bceda87c24473f authored almost 7 years ago by Ruben Bridgewater <[email protected]>
Instead of exposing internals of async_hooks & async_wrap throughout
the code base, create neces...
PR-URL: https://github.com/nodejs/node/pull/18788
Refs: https://github.com/nodejs/node/pull/1872...
Refs: https://github.com/nodejs/node/pull/18769
PR-URL: https://github.com/nodejs/node/pull/187...
61e3e6d56feefcd88f2baeb59c31c327835a8d90 authored almost 7 years ago by Gibson Fahnestock <[email protected]>
PR-URL: https://github.com/nodejs/node/pull/18775
Reviewed-By: Weijia Wang <[email protected]>
R...
PR-URL: https://github.com/nodejs/node/pull/18775
Reviewed-By: Weijia Wang <[email protected]>
R...
This enables the eslint dot-notation rule for all code instead of
only in /lib.
PR-URL: https:/...
463d1a490f2ed4679f8805fa57084b4445ea40d6 authored almost 7 years ago by Ruben Bridgewater <[email protected]>Refs: https://github.com/v8/v8/compare/6.4.388.44...6.4.388.45
PR-URL: https://github.com/nodej...
36386dc4e349eb593208820949a02e93dc1ffd87 authored almost 7 years ago by Myles Borins <[email protected]>
PR-URL: https://github.com/nodejs/node/pull/18798
Reviewed-By: Richard Lau <[email protected]>
R...
- add new constants
- replace "magic" numbers in "module" by constants
PR-URL: https://github.c...
c86fe511f4924ebbf7291fd938e5cd952199145b authored almost 7 years ago by Sergey Golovin <[email protected]>
PR-URL: https://github.com/nodejs/node/pull/18784
Reviewed-By: James M Snell <[email protected]>...
In the spirit of [17399](https://github.com/nodejs/node/pull/17399),
we can also simplify checkI...
* exit naturally, don't use process.exit()
* ensure callbacks are actually called
PR-URL: https...
3c29adb84fac00dfc16c51725e4832789610c6a7 authored almost 7 years ago by Ben Noordhuis <[email protected]>The `millions` argument was missing.
PR-URL: https://github.com/nodejs/node/pull/18787
Reviewed...
PR-URL: https://github.com/nodejs/node/pull/18257
Reviewed-By: Joyee Cheung <[email protected]...
`readable` and `writable` properties can be passed directly to the
`net.Socket` constructor. Thi...
PR-URL: https://github.com/nodejs/node/pull/18822
Reviewed-By: Ruben Bridgewater <ruben@bridgewa...
This adds a implicit common.mustCall to the callback provided to
the countdown.
PR-URL: https:/...
941bc93f2229d4a28c3d3246878c7fd4fbf443ec authored almost 7 years ago by Bamieh <[email protected]>
Number.isNaN is now as fast as `val !== val`. Switch to the more
readable version. Also switch a...
These two NaN entries are not necessary and we can safely remove them.
PR-URL: https://github.c...
43b8ce4ce741a8a6a435e4073ca5a9ec188d4515 authored almost 7 years ago by Ruben Bridgewater <[email protected]>
PR-URL: https://github.com/nodejs/node/pull/18543
Reviewed-By: Luigi Pinca <[email protected]...
PR-URL: https://github.com/nodejs/node/pull/18543
Reviewed-By: Luigi Pinca <[email protected]...
Wrap SafeGetenv() in util binding with the purpose of protecting
the cases when env vars are acc...
When building the node with `--shared` option, the major output is the
shared library. However, ...
PR-URL: https://github.com/nodejs/node/pull/18650
Reviewed-By: Ben Noordhuis <[email protected]...
This was never documented and the `assert` module should be used
instead.
PR-URL: https://githu...
703e37cf3fb163a4630a10f32bf5d5ec35142019 authored almost 7 years ago by Ruben Bridgewater <[email protected]>
PR-URL: https://github.com/nodejs/node/pull/18818
Refs: https://github.com/nodejs/node/commit/ef...
Prohibit the usage of `assert.doesNotThrow()`.
PR-URL: https://github.com/nodejs/node/pull/1866...
15bb8437fd0a0eb07845ccf746c9ecb47fcfcb25 authored almost 7 years ago by Ruben Bridgewater <[email protected]>
Add punctuation and comments about code that should not throw.
Also remove a obsolete test and r...
There is actually no reason to use `assert.doesNotThrow()` in the
tests. If a test throws, just ...
This enables the `no-unsafe-finally` eslint rule to make sure we
have a proper control flow in t...
PR-URL: https://github.com/nodejs/node/pull/18757
Reviewed-By: Ruben Bridgewater <ruben@bridgewa...
Move the print statements below a console.log call.
PR-URL: https://github.com/nodejs/node/pull...
759a083d4497b6300be720029cb5b2004e4637b1 authored almost 7 years ago by Ruben Bridgewater <[email protected]>
This commit set the Environment as a thread local when CreateEnvironment
is called which is curr...
The legacy MakeCallback functions do not provide a mechanism to
propagate async context. This me...
Remove unnecessary condition in lib/module.js.
Refs: https://github.com/nodejs/node/pull/18593#...
3b9cc424a4240c10003dca71c5a7478232e3d0d8 authored almost 7 years ago by Rich Trott <[email protected]>
SetAutorunMicrotasks is deprecated and a warning will be emitted
starting from V8 6.5. Use the n...
PR-URL: https://github.com/nodejs/node/pull/18211
Reviewed-By: James M Snell <[email protected]>...
Adding ServerResponse.writeProcessing to send 102 status codes.
Added an `'information'` event ...
baf84950781f253ab0244779a9e40b5dc0f4930e authored almost 7 years ago by Miles Elam <[email protected]>
A subroutine does not work as a replacement for the `python` command
since one cannot use a subr...
PR-URL: https://github.com/nodejs/node/pull/18783
Reviewed-By: James M Snell <[email protected]>...
Update the dead link to <keygen> documentation.
Add a link to mozilla developper documentation b...
PR-URL: https://github.com/nodejs/node/pull/17640
Reviewed-By: Anna Henningsen <[email protected]...
In a conversation Fedor requested that this PR be made. They plan
to continue working on core as...
This commit adds a regression test for
de848ac1e0483327a2ce8716c3f8567eaeacb660, which broke
mul...
The timer in NAPI's test_callback_scope/test-resolve-async.js
can be removed. If the test fails,...
This adds support for ensuring that the top-level main into Node is
supported loading when it ha...
Encapsulate stream requests more:
- `WriteWrap` and `ShutdownWrap` classes are now tailored to ...
0e7b61229aa602e55c5fb034a63d7da97eecff3b authored almost 7 years ago by Anna Henningsen <[email protected]>
PR-URL: https://github.com/nodejs/node/pull/18676
Reviewed-By: Ben Noordhuis <[email protected]...
Otherwise `this[kCurrentWriteRequest]` is set to a value even
if one of the `write` calls throws...
Use `DoTryWrite()` to write data to the underlying socket.
This does probably not make any diffe...
PR-URL: https://github.com/nodejs/node/pull/18642
Reviewed-By: Jon Moss <[email protected]>
Rev...
Redefining private breaks any private inheritance in the
included files. We can simply declare G...
Currently when building --without-ssl a 'ERR_NO_CRYPTO' error is
reported.
This is not currentl...
e9ba0cfd46d75d21e9e2359e40710c3ee46a296a authored almost 7 years ago by Daniel Bevenius <[email protected]>
This was not caught by the linter because the release commit
for 6.13.0 came from a different br...
This LTS release comes with 112 commits, 17 of which are considered
Semver-Minor. This includes ...
There are libraries which invoke StringDecoder using .call and
.inherits, which directly conflic...
PR-URL: https://github.com/nodejs/node/pull/18687
Refs: https://github.com/v8/v8/compare/6.4.388...
PR-URL: https://github.com/nodejs/node/pull/18654
Reviewed-By: Richard Lau <[email protected]>
R...
PR-URL: https://github.com/nodejs/node/pull/18740
Reviewed-By: Joyee Cheung <[email protected]...
PR-URL: https://github.com/nodejs/node/pull/18734
Fixes: https://github.com/nodejs/node/issues/1...
Add unexpected value to dgram socket type assertion and fix the size
entry.
PR-URL: https://git...
18e728474031911650992fee9cd113be0d00d67e authored almost 7 years ago by Kevin Caulfield <[email protected]>
Add useful info about process.domain to error meesages in the
uncaughtException event listener a...