GeezerVison is showing github.com — v15.6.0 proposal by danielleadams · Pull Request #36889 · nodejs/node
Conversation
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.
bricss and dnlup reacted with heart emoji
bricss, ExE-Boss, and dnlup reacted with rocket emoji
All reactions
-
❤️
2 reactions
-
🚀
3 reactions
* Replace unused identifier with Python convention `_`.
* Remove unneeded parentheses.
* Remove line-wrapping.
I confirmed that this doesn't change the output by running `./configure
--shared-zlib` and confirming that it created the same `common.gypi`
with and without these changes. The code changed here doesn't run unless
there is a `--shared-*` flag.
PR-URL: #36551
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Daijiro Wachi <[email protected]>
I was working on AbortSignal for spawn and noticed there is a leak in
the current code for AbortSignal support in child_process since it
removes the wrong listener. I used the new signal as argument feature
to make removing the listener easier and added a test.
PR-URL: #36424
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
This commit adds a suggestion to redirect stderr for
EnvironmentTest.EnvironmentWithNoESMLoader.
The motivation for this is that currently this tests prints the
following error (which is expected):
vm:module(0):1
globalThis.importResult = import("")
^
Error: Not supported
at vm:module(0):1:1
at SourceTextModule.evaluate (node:internal/vm/module:229:23)
at node:embedder_main_12:1:328
at processTicksAndRejections (node:internal/process/task_queues:93:5)
It might not be obvious which test caused this error just by looking at
the output above and it would be nice if it was not displayed.
PR-URL: #36548
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Joyee Cheung <[email protected]>
We use `is None` instead of `== None` everywhere (which mostly just
means test.py) except in one place in genv8constants.py. Switch to `is
None` in genv8constants.py. This is slightly more efficient, although I
can't imagine that makes a measurable difference here.
PR-URL: #36606
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Using an explicit constructor is necessary to avoid relying on
`Array.prototype[Symbol.iterator]` and `%ArrayIteratorPrototype%.next`,
which can be mutated by users.
PR-URL: #36587
Refs: #36428
Refs: #36532
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Adds a simple benchmark for https server based on the http simple
benchmark. Updates benchmarker integration for autocannon and wrk,
so that they support https scheme.
Also adds a new HTTPS section and improves HTTP/2 section in
the benchmark guide.
PR-URL: #36612
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Antoine du Hamel <[email protected]>
danielleadams
added a commit
that referenced
this pull request
Jan 13, 2021
PR-URL: #36889
Notable changes:
* child_process:
* add 'overlapped' stdio flag (Thiago Padilha) [#29412](#29412)
* support AbortSignal in fork (Benjamin Gruenbaum) [#36603](#36603)
* crypto:
* implement basic secure heap support (James M Snell) [#36779](#36779)
* fixup bug in keygen error handling (James M Snell) [#36779](#36779)
* introduce X509Certificate API (James M Snell) [#36804](#36804)
* implement randomuuid (James M Snell) [#36729](#36729)
* doc:
* update release key for Danielle Adams (Danielle Adams) [#36793](#36793)
* add dnlup to collaborators (Daniele Belardi) [#36849](#36849)
* add panva to collaborators (Filip Skokan) [#36802](#36802)
* add yashLadha to collaborator (Yash Ladha) [#36666](#36666)
* http:
* set lifo as the default scheduling strategy in Agent (Matteo Collina) [#36685](#36685)
* net:
* support abortSignal in server.listen (Nitzan Uziely) [#36623](#36623)
* process:
* add direct access to rss without iterating pages (Adrien Maret) [#34291](#34291)
* v8:
* fix native constructors (ExE Boss) [#36549](#36549)
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Commit message for cf44a24 reads 2020 rather than 2021 (it does still feel like it). Shouldn't be necessary to rerun all of the CIs if that's the only update though
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.
BethGriggs reacted with heart emoji
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.
danielleadams
added a commit
that referenced
this pull request
Jan 13, 2021
PR-URL: #36889
Notable changes:
* child_process:
* add 'overlapped' stdio flag (Thiago Padilha) [#29412](#29412)
* support AbortSignal in fork (Benjamin Gruenbaum) [#36603](#36603)
* crypto:
* implement basic secure heap support (James M Snell) [#36779](#36779)
* fixup bug in keygen error handling (James M Snell) [#36779](#36779)
* introduce X509Certificate API (James M Snell) [#36804](#36804)
* implement randomuuid (James M Snell) [#36729](#36729)
* doc:
* update release key for Danielle Adams (Danielle Adams) [#36793](#36793)
* add dnlup to collaborators (Daniele Belardi) [#36849](#36849)
* add panva to collaborators (Filip Skokan) [#36802](#36802)
* add yashLadha to collaborator (Yash Ladha) [#36666](#36666)
* http:
* set lifo as the default scheduling strategy in Agent (Matteo Collina) [#36685](#36685)
* net:
* support abortSignal in server.listen (Nitzan Uziely) [#36623](#36623)
* process:
* add direct access to rss without iterating pages (Adrien Maret) [#34291](#34291)
* v8:
* fix native constructors (ExE Boss) [#36549](#36549)
This comment has been minimized.
This comment has been minimized.
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.
danielleadams
added a commit
that referenced
this pull request
Jan 14, 2021
PR-URL: #36889
Notable changes:
* child_process:
* add 'overlapped' stdio flag (Thiago Padilha) [#29412](#29412)
* support AbortSignal in fork (Benjamin Gruenbaum) [#36603](#36603)
* crypto:
* implement basic secure heap support (James M Snell) [#36779](#36779)
* fixup bug in keygen error handling (James M Snell) [#36779](#36779)
* introduce X509Certificate API (James M Snell) [#36804](#36804)
* implement randomuuid (James M Snell) [#36729](#36729)
* doc:
* update release key for Danielle Adams (Danielle Adams) [#36793](#36793)
* add dnlup to collaborators (Daniele Belardi) [#36849](#36849)
* add panva to collaborators (Filip Skokan) [#36802](#36802)
* add yashLadha to collaborator (Yash Ladha) [#36666](#36666)
* http:
* set lifo as the default scheduling strategy in Agent (Matteo Collina) [#36685](#36685)
* net:
* support abortSignal in server.listen (Nitzan Uziely) [#36623](#36623)
* process:
* add direct access to rss without iterating pages (Adrien Maret) [#34291](#34291)
* v8:
* fix native constructors (ExE Boss) [#36549](#36549)
|
Tests are passing 🎉 so this is ready to go. (@nodejs/releasers)
I am running a citgm smoker on branch v15.x to check for new errors from packages, but it looks like a lot of the errors are from packages that have not passed on previous 15.x releases.
|
simov reacted with rocket emoji
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.
PR-URL: #36889
Notable changes:
* child_process:
* add 'overlapped' stdio flag (Thiago Padilha) (#29412)
* support AbortSignal in fork (Benjamin Gruenbaum) (#36603)
* crypto:
* implement basic secure heap support (James M Snell) (#36779)
* fixup bug in keygen error handling (James M Snell) (#36779)
* introduce X509Certificate API (James M Snell) (#36804)
* implement randomuuid (James M Snell) (#36729)
* doc:
* update release key for Danielle Adams (Danielle Adams) (#36793)
* add dnlup to collaborators (Daniele Belardi) (#36849)
* add panva to collaborators (Filip Skokan) (#36802)
* add yashLadha to collaborator (Yash Ladha) (#36666)
* http:
* set lifo as the default scheduling strategy in Agent (Matteo Collina) (#36685)
* net:
* support abortSignal in server.listen (Nitzan Uziely) (#36623)
* process:
* add direct access to rss without iterating pages (Adrien Maret) (#34291)
* v8:
* fix native constructors (ExE Boss) (#36549)
danielleadams
added a commit
that referenced
this pull request
Jan 15, 2021
PR-URL: #36889
Notable changes:
* child_process:
* add 'overlapped' stdio flag (Thiago Padilha) (#29412)
* support AbortSignal in fork (Benjamin Gruenbaum) (#36603)
* crypto:
* implement basic secure heap support (James M Snell) (#36779)
* fixup bug in keygen error handling (James M Snell) (#36779)
* introduce X509Certificate API (James M Snell) (#36804)
* implement randomuuid (James M Snell) (#36729)
* doc:
* update release key for Danielle Adams (Danielle Adams) (#36793)
* add dnlup to collaborators (Daniele Belardi) (#36849)
* add panva to collaborators (Filip Skokan) (#36802)
* add yashLadha to collaborator (Yash Ladha) (#36666)
* http:
* set lifo as the default scheduling strategy in Agent (Matteo Collina) (#36685)
* net:
* support abortSignal in server.listen (Nitzan Uziely) (#36623)
* process:
* add direct access to rss without iterating pages (Adrien Maret) (#34291)
* v8:
* fix native constructors (ExE Boss) (#36549)
Uh oh!
There was an error while loading. Please reload this page.
Assignees
No one assigned
Labels
build
Issues and PRs related to Node.js builds or CI infrastructure.
doc
Issues and PRs related to Node.js documentation.
http2
Issues and PRs related to the http2 subsystem.
meta
Issues and PRs related to the general management of the project.
tools
Issues and PRs related to the tools directory.
Milestone
No milestone
Development
Successfully merging this pull request may close these issues.
Uh oh!
There was an error while loading. Please reload this page.
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
|