addaleax · GitHub

@addaleax added buffer

Issues and PRs related to the buffer subsystem.

semver-major

PRs that contain breaking changes and should be released in the next major version.

labels

Mar 22, 2018

@addaleax

@nodejs-github-bot nodejs-github-bot added buffer

Issues and PRs related to the buffer subsystem.

util

Issues and PRs related to the built-in util module.

labels

Mar 22, 2018

@addaleax addaleax added notable-change

PRs with changes that should be highlighted in changelogs.

and removed util

Issues and PRs related to the built-in util module.

labels

Mar 22, 2018

@addaleax

vsemozhetbyt

ChALkeR

ChALkeR

seishun

ChALkeR

ChALkeR

mafintosh

benjamingr

jdalton

mcollina

@Trott Trott added the tsc-agenda

Issues and PRs to discuss during the meetings of the TSC.

label

Apr 3, 2018

mhdawson

evanlucas

MylesBorins

@Trott Trott removed the tsc-agenda

Issues and PRs to discuss during the meetings of the TSC.

label

Apr 4, 2018
In addition to `--pending-deprecation`, emit a deprecation warning
for usage of the `Buffer()` constructor for call sites that are outside
of `node_modules`.
The goal of this is to better target developers, rather than
burdening users with an omnipresent and quickly ignored warning.
This implements the result of a TSC meeting discussion
from March 22, 2018.
Refs: nodejs#19079 (comment)
PR-URL: nodejs#19524
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Rod Vagg <rod@vagg.org>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Evan Lucas <evanlucas@me.com>
Reviewed-By: Myles Borins <myles.borins@gmail.com>

@addaleax

@BridgeAR BridgeAR added the author ready

PRs that have at least one approval, no pending requests for changes, and a CI started.

label

Apr 9, 2018

BridgeAR pushed a commit to BridgeAR/node that referenced this pull request

Apr 9, 2018
In addition to `--pending-deprecation`, emit a deprecation warning
for usage of the `Buffer()` constructor for call sites that are outside
of `node_modules`.
The goal of this is to better target developers, rather than
burdening users with an omnipresent and quickly ignored warning.
This implements the result of a TSC meeting discussion
from March 22, 2018.
PR-URL: nodejs#19524
Refs: nodejs#19079 (comment)
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Rod Vagg <rod@vagg.org>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Evan Lucas <evanlucas@me.com>
Reviewed-By: Myles Borins <myles.borins@gmail.com>

Closed

4 tasks

Closed

addaleax added a commit to addaleax/node that referenced this pull request

Apr 14, 2018
Taking the source code of a function and running it in another
context does not play well with coverage instrumentation.
For now, do the simple thing and just write the source code as
a string literal.
Fixes: nodejs#19912
Refs: nodejs#19524

Closed

2 tasks

addaleax added a commit that referenced this pull request

Apr 15, 2018
Taking the source code of a function and running it in another
context does not play well with coverage instrumentation.
For now, do the simple thing and just write the source code as
a string literal.
Fixes: #19912
Refs: #19524
PR-URL: #20035
Reviewed-By: Tiancheng "Timothy" Gu <timothygu99@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>

jasnell pushed a commit that referenced this pull request

Apr 16, 2018
Taking the source code of a function and running it in another
context does not play well with coverage instrumentation.
For now, do the simple thing and just write the source code as
a string literal.
Fixes: #19912
Refs: #19524
PR-URL: #20035
Reviewed-By: Tiancheng "Timothy" Gu <timothygu99@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>

Closed

4 tasks

Closed

4 tasks

Read the original on github.com ↗