V5.10.1 proposal #6060
Merged
V5.10.1 proposal #6060
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Refactor unused self=this code to code without without this pattern making it more consistent with the rest of our code. PR-URL: #5857 Reviewed-By: Brian White <mscdex@mscdex.net> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com> Reviewed-By: Roman Klauke <romankl@users.noreply.github.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> PR-URL: #5936
`test-stdtout-close-unref.js` will fail if `process.stdin._handle` does
not exist. On UNIX-like operating systems, you can see this failure this
way:
./node test/parallel/test-stdout-close-unref.js < /dev/null
This issue has been experienced by @bengl and @drewfish in a Docker
container. I'm not sure why they are experiencing it in their
environment, but since it is possible that the `_handle` property does
not exist, let's use `child_process.spawn()` to make sure it exists.
PR-URL: #5916
Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
IPv6 addresses in Host header (URI), must be enclosed within square brackets, in order to properly separate the host address from any port reference. PR-URL: #5314 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Evan Lucas <evanlucas@me.com>
Introduce two new targets we will populate with actions once merged into all branches we need to support through CI. PR-URL: #5921 Reviewed-By: Brian White <mscdex@mscdex.net> Reviewed-By: Myles Borins <myles.borins@gmail.com> Reviewed-By: João Reis <reis@janeasystems.com> Reviewed-By: James M Snell <jasnell@gmail.com>
Remove timer to avoid the test timing out occasionally. PR-URL: #5676 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Claudio Rodriguez <cjrodr@yahoo.com>
Rather than attempting to keep two versions of docs for timers up to date, keep them in timers.markdown, and leave references to them in globals.markdown. Add setImmediate and clearImmediate to globals.markdown. Change "To schedule" to "Schedules" in timers.markdown. PR-URL: #5837 Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com> Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com> Reviewed-By: Claudio Rodriguez <cjrodr@yahoo.com> Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com>
`require-buffer` is only enabled in the `lib` directory. There is no need to disable it in `test`. PR-URL: #6013 Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Includes cli.markdown in all.markdown PR-URL: #6011 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
PR-URL: #5986 Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Use empty string instead of `www.google.com` for tests where we are just doing parameter evaluation. This will avoid DNS lookups which appear to be causing flakiness on Raspberry Pi devices in CI. PR-URL: #5996 Fixes: #5554 Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
PR-URL: #6029 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
Provide encrypted links to freenode IRC. PR-URL: #6019 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Roman Klauke <romaaan.git@gmail.com> Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
The test intentionally assigns a global. Use `global` namespace to make it clear that it is intentional and not an accidental leak. PR-URL: #6026 Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
The value is retrieved from `process.arch` in node itself. PR-URL: #5997 Reviewed-By: João Reis <reis@janeasystems.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Johan Bergström <bugs@bergstroem.nu>
There was a comment in `test-domain-crypto.js` indicating that the pollution of the `global` object with a `domain` property was intentional. Provide more information in the comment so someone may easily determine why. Use `global.domain` rather than declaring `domain` without the `var` keyword to more clearly signal that the pollution is intentional. PR-URL: #6017 Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Use `global` to be explicit that a global variable is intended. PR-URL: #6014 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
Do not convert typed arrays to `Buffer` for deepEqual since their values may not be accurately represented by 8-bit ints. Instead perform binary comparison of underlying `ArrayBuffer`s, but only when the array types match. Never apply any kind of optimization for floating-point typed arrays since bit pattern equality is not the right kind of check for them. PR-URL: #5910 Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com> Fixes: #5907
All JS files have been moved out of the `src` directory so the `.eslintrc` file in that directory can also be removed. PR-URL: #5959 Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Roman Klauke <romankl@users.noreply.github.com>
this adds an example of a long running node process that actually executes node code. Also it mentions the not to harmonic detach behaviours of the different platforms, whereas detaching on unix requires ignoring the child_process' stdio explicitely. PR-URL: #5330 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
MylesBorins
added
the
meta
|
ci: https://ci.nodejs.org/job/node-test-pull-request/2166/ citgm: https://ci.nodejs.org/job/thealphanerd-smoker/183/ edit: re running arm fanned tests --> https://ci.nodejs.org/job/node-test-commit-arm-fanned/2131/ arm fanned looks good... moving forwad with release |
|
LGTM |
|
It looks like the notable changes section has a formatting issue, |
MylesBorins
force-pushed
the
v5.10.1-proposal
branch
from
6888706
to
c53141b
Compare
Notable changes: http: * Enclose IPv6 Host header in square brackets. This will enable proper seperation of the host adress from any port reference (Mihai Potra) #5314 path: * Make win32.isAbsolute more consistent (Brian White) #6028 PR-URL: #6060 Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
MylesBorins
force-pushed
the
v5.10.1-proposal
branch
from
c53141b
to
5c4a414
Compare
|
Good eye. Updated
|
MylesBorins
pushed a commit
that referenced
this pull request
Apr 5, 2016
Notable changes: http: * Enclose IPv6 Host header in square brackets. This will enable proper seperation of the host adress from any port reference (Mihai Potra) #5314 path: * Make win32.isAbsolute more consistent (Brian White) #6028 PR-URL: #6060 Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
2016-04-05, Version 5.10.1 (Stable), @thealphanerd
Notable changes
http:
path:
Commits
0f5a51ae4b] - assert: Check typed array view type in deepEqual (Anna Henningsen) #5910e966d1f5db] - buffer: don't setkNoZeroFillflag in allocUnsafe (Vladimir Kurchatkin) #60073f75751c2e] - build: introduce ci targets for lint/benchmark (Johan Bergström) #5921781290b61d] - doc: refine child_process detach behaviour (Robert Jefe Lindstaedt) #5330aa9fb03202] - doc: use HTTPS for links where possible (Rich Trott) #6019dd25984838] - doc: note assert.throws() pitfall (Rich Trott) #6029f879f5e68a] - doc: document unspecified behavior for buf.write* methods (James M Snell) #5925f12c3861e0] - doc: clarify stdout/stderr arguments to callback (James M Snell) #6015ce173716be] - doc: add 'Command Line Options' to 'View on single page' (firedfox) #60117337ef6422] - doc: minor argument formatting in stream.markdown (James M Snell) #60160ae5d027c6] - doc: clarify that __dirname is module local (James M Snell) #60188bec8aa41f] - doc: consolidate timers docs in timers.markdown (Bryan English) #58370a13099c42] - etw: add event messages (João Reis) #5936c6ac6f2ea1] - http: Corrects IPv6 address in Host header (Mihai Potra) #53148317778925] - meta: add "joining a wg" section to WORKING_GROUPS.md (Matteo Collina) #5488f3f19ee5e2] - net: refactor self=this to arrow functions (Benjamin Gruenbaum) #58571c4007927d] - path: fix win32.isAbsolute() inconsistency (Brian White) #6028059b607a4f] - test: make use of globals explicit (Rich Trott) #6014cc8fcc5a07] - test: be explicit about polluting ofglobal(Rich Trott) #60177db7a820b9] - test: make arch available in status files (Santiago Gimeno) #599702f2ebd9b4] - test: explicitly set global in test-repl (Rich Trott) #60262ab1237137] - test: fix flaky test-net-socket-timeout-unref (Rich Trott) #60030127c2bd39] - test: fix test-dns.js flakiness (Rich Trott) #59966052ced37f] - test: fix error message checks in test-module-loading (James M Snell) #5986a40b0cb673] - test: refactor http-end-throw-socket-handling (Santiago Gimeno) #567696bb315262] - test: ensure _handle property existence (Rich Trott) #59164f1fa2adeb] - test: fix offending max-len linter error (Sakthipriyan Vairamani) #5980f14d71ccea] - test: stdin is not always a net.Socket (Jeremiah Senkpiel) #593550a062e691] - tools: remove obsolete lint config file (Rich Trott) #59597491fdcfe9] - tools: remove disabling of already-disabled rule (Rich Trott) #6013