Skip to content
This repository was archived by the owner on Aug 11, 2022. It is now read-only.

test: don't use undocumented res.writeHeader alias#15874

Merged
zkat merged 1 commit intonpm:release-nextfrom
ChALkeR:patch-1
Mar 8, 2017
Merged

test: don't use undocumented res.writeHeader alias#15874
zkat merged 1 commit intonpm:release-nextfrom
ChALkeR:patch-1

Conversation

@ChALkeR
Copy link
Copy Markdown
Contributor

@ChALkeR ChALkeR commented Feb 26, 2017

res.writeHeader is an undocumented alias to res.writeHead
It's docs-deprecated in nodejs/node#11355

Refs: nodejs/node#11355

Note: I haven't run tests locally and judging by the folder this resides in I am not sure if this is covered on CI.


CI build failure is due to a broken Slack integration:

Error sending Slack notification: Value cannot be null.
Parameter name: Either incoming webhook URL or Slack authentication token is required.

res.writeHeader is an undocumented alias to res.writeHead
It's docs-deprecated in nodejs/node#11355
@iarna iarna added this to the next milestone Mar 6, 2017
Copy link
Copy Markdown
Contributor

@zkat zkat left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thanks, @ChALkeR :)

@zkat zkat changed the base branch from latest to release-next March 8, 2017 01:57
@zkat zkat merged commit 6121ab6 into npm:release-next Mar 8, 2017
zkat pushed a commit that referenced this pull request Mar 9, 2017
res.writeHeader is an undocumented alias to res.writeHead
It's docs-deprecated in nodejs/node#11355

PR-URL: #15874
Credit: @ChALkeR 
Reviewed-By: @zkat
zkat pushed a commit that referenced this pull request Mar 10, 2017
res.writeHeader is an undocumented alias to res.writeHead
It's docs-deprecated in nodejs/node#11355

PR-URL: #15874
Credit: @ChALkeR 
Reviewed-By: @zkat
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants