GeezerVison is showing github.com — v15.12.0 proposal by danielleadams · Pull Request #37766 · nodejs/node
Conversation
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.
bricss reacted with thumbs up emoji
bricss reacted with hooray emoji
bricss reacted with heart emoji
bricss reacted with rocket emoji
All reactions
-
👍
1 reaction
-
🎉
1 reaction
-
❤️
1 reaction
-
🚀
1 reaction
Improve the fsPromises readFile performance
by allocating only one buffer, when size is known,
increase the size of the readbuffer chunks,
and dont read more data if size bytes have been read
Refs: #37583
PR-URL: #37608
Reviewed-By: Benjamin Gruenbaum <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Anna Henningsen <[email protected]>
Many anonymous functions use the empty string as their name.
Since the DEP0097 logic was using nullish coalescing, these
functions were not being displayed as anonymous. This commit
updates the logic to use || instead of ??.
PR-URL: #37550
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Gerhard Stöbich <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Iterating over arrays should be avoided because it relies on
user-mutable global methods (`Array.prototype[Symbol.iterator]`
and `%ArrayIteratorPrototype%.next`), we should instead use
other alternatives. This commit adds a rule that disallow
array destructuring syntax in favor of object destructuring syntax.
Note that you can ignore this rule if you are using
the array destructuring syntax over a safe iterable, or
actually want to iterate over a user-provided object.
PR-URL: #36818
Reviewed-By: Rich Trott <[email protected]>
Add `ev.ports` for spec compliancy.
Since we only emit the raw `data` value, and only create the
`MessageEvent` instance if there are EventTarget-style listeners,
we store the ports list temporarily on the MessagePort object itself,
so that we can look it up when we need to create the event object.
Fixes: #37358
PR-URL: #37538
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Joyee Cheung <[email protected]>
* Since exec calls execFile and execFile internally calls spawn with
options.signal, the signal parameter has been documented under exec
as well.
* Refactor the description of signal under all the functions.
* Add examples of usage of signal under all the functions and add
missing requires in the other examples.
PR-URL: #37528
Reviewed-By: Benjamin Gruenbaum <[email protected]>
Reviewed-By: Antoine du Hamel <[email protected]>
Install `eslint-plugin-markdown` at the same level of `eslint` without
cd'ing into `eslint` directory, otherwise the following error is raised:
```
npm ERR! code ERESOLVE
npm ERR! Cannot destructure property 'name' of 'node' as it is null.
```
PR-URL: #37566
Fixes: #37560
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Ruy Adorno <[email protected]>
Reviewed-By: Antoine du Hamel <[email protected]>
error: comparison of integer expressions of different signedness:
‘int’ and ‘uint64_t’ {aka ‘long unsigned int’} [-Werror=sign-compare]
2562 | if (!success || std::numeric_limits<int32_t>::max() <
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^
2563 | token_start_internal_value_) {
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~
PR-URL: #37573
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Benjamin Gruenbaum <[email protected]>
Since `rr` and `shared` both belongs to the same prototype declaration
and differes only in the handler declaration, this can be abstracted to
a same type of function arguments passing.
PR-URL: #36610
Reviewed-By: James M Snell <[email protected]>
This comment has been minimized.
|
@nodejs/releasers this is ready for review!
|
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.
This comment has been minimized.
danielleadams
added a commit
that referenced
this pull request
Mar 17, 2021
PR-URL: #37766
Notable changes:
* crypto:
* add optional callback to crypto.sign and crypto.verify (Filip Skokan) #37500
* support JWK objects in create\*Key (Filip Skokan) #37254
* deps:
* switch openssl to quictls/openssl (James M Snell) #37601
* fs:
* improve fsPromises writeFile performance (Nitzan Uziely) #37610
* improve fsPromises readFile performance (Nitzan Uziely) #37608
* lib:
* implement AbortSignal.abort() (James M Snell) #37693
* node-api:
* define version 8 (Gabriel Schulhof) #37652
* worker:
* add setEnvironmentData/getEnvironmentData (James M Snell) #37486
This comment has been minimized.
This comment has been minimized.
|
Maybe it's still possible to pull that in #37712? 🙄
|
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.
PR-URL: #37766
Notable changes:
* crypto:
* add optional callback to crypto.sign and crypto.verify (Filip Skokan) #37500
* support JWK objects in create\*Key (Filip Skokan) #37254
* deps:
* update to [email protected] (Guy Bedford) #37712
* switch openssl to quictls/openssl (James M Snell) #37601
* fs:
* improve fsPromises writeFile performance (Nitzan Uziely) #37610
* improve fsPromises readFile performance (Nitzan Uziely) #37608
* lib:
* implement AbortSignal.abort() (James M Snell) #37693
* node-api:
* define version 8 (Gabriel Schulhof) #37652
* worker:
* add setEnvironmentData/getEnvironmentData (James M Snell) #37486
bricss reacted with heart emoji
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.
This comment has been minimized.
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.
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
Mar 17, 2021
PR-URL: #37766
Notable changes:
* crypto:
* add optional callback to crypto.sign and crypto.verify (Filip Skokan) #37500
* support JWK objects in create\*Key (Filip Skokan) #37254
* deps:
* update to [email protected] (Guy Bedford) #37712
* switch openssl to quictls/openssl (James M Snell) #37601
* fs:
* improve fsPromises writeFile performance (Nitzan Uziely) #37610
* improve fsPromises readFile performance (Nitzan Uziely) #37608
* lib:
* implement AbortSignal.abort() (James M Snell) #37693
* node-api:
* define version 8 (Gabriel Schulhof) #37652
* worker:
* add setEnvironmentData/getEnvironmentData (James M Snell) #37486
targos
removed
build
Issues and PRs related to Node.js builds or CI infrastructure.
doc
Issues and PRs related to Node.js documentation.
needs-ci
PRs that need a full CI run.
npm
Issues and PRs related to the npm client dependency or the npm registry.
tools
Issues and PRs related to the tools directory.
meta
Issues and PRs related to the general management of the project.
labels
Jun 6, 2021
Reviewers
+1 more reviewer
targos
targos approved these changes
Reviewers whose approvals may not affect merge requirements
Uh oh!
There was an error while loading. Please reload this page.
Assignees
No one assigned
Labels
release
Issues and PRs related to Node.js releases.
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.
|