New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test: replace Function with Arrow Function to test-http-pause.js #17306

Closed
wants to merge 1 commit into
base: master
from

Conversation

@kjunichi
Contributor

kjunichi commented Nov 26, 2017

This is part of Nodefest's Code and Learn nodejs/code-and-learn#72

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • tests and/or benchmarks are included
  • documentation is changed or added
  • commit message follows commit guidelines
Affected core subsystem(s)

test

@yosuke-furukawa

This comment has been minimized.

Show comment
Hide comment
@yosuke-furukawa

yosuke-furukawa Nov 26, 2017

Member

Please remove your package-lock.json

Member

yosuke-furukawa commented Nov 26, 2017

Please remove your package-lock.json

@MylesBorins MylesBorins referenced this pull request Nov 26, 2017

Closed

Add remark deps #17315

@targos

targos approved these changes Nov 26, 2017

@maclover7

This comment has been minimized.

Show comment
Hide comment

@gibfahn gibfahn referenced this pull request Nov 26, 2017

Closed

Add remark dependencies #17320

@kjunichi

This comment has been minimized.

Show comment
Hide comment
@kjunichi

kjunichi Nov 27, 2017

Contributor

sorry for late reply.

@yosuke-furukawa thank you for your advise.

I've removed follwing files:

  • tools/remark-cli/package-lock.json
  • tools/remark-preset-lint-node/package-lock.json

But it seems CI is not OK.
What shoud I do?

Contributor

kjunichi commented Nov 27, 2017

sorry for late reply.

@yosuke-furukawa thank you for your advise.

I've removed follwing files:

  • tools/remark-cli/package-lock.json
  • tools/remark-preset-lint-node/package-lock.json

But it seems CI is not OK.
What shoud I do?

@maclover7

This comment has been minimized.

Show comment
Hide comment
@maclover7

maclover7 Nov 28, 2017

Member

@kjunichi No worries -- it looks like linuxone hit a random failure -- rerunning at https://ci.nodejs.org/job/node-test-commit-linuxone/10706/nodes=rhel72-s390x, and then should be good to land :)

Member

maclover7 commented Nov 28, 2017

@kjunichi No worries -- it looks like linuxone hit a random failure -- rerunning at https://ci.nodejs.org/job/node-test-commit-linuxone/10706/nodes=rhel72-s390x, and then should be good to land :)

@maclover7

This comment has been minimized.

Show comment
Hide comment
@maclover7

maclover7 Nov 28, 2017

Member

Yup, random failure -- CI is all green, landing...

Member

maclover7 commented Nov 28, 2017

Yup, random failure -- CI is all green, landing...

@maclover7 maclover7 self-assigned this Nov 28, 2017

maclover7 added a commit that referenced this pull request Nov 28, 2017

test: replace function with ES6 arrow function
PR-URL: #17306
Reviewed-By: Daijiro Wachi <daijiro.wachi@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Jon Moss <me@jonathanmoss.me>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Yuta Hiroto <hello@about-hiroppy.com>
@maclover7

This comment has been minimized.

Show comment
Hide comment
@maclover7

maclover7 Nov 28, 2017

Member

Landed in a177708, congrats on your first PR to Node.js!
❤️ 💚 💙 💛 💜

Member

maclover7 commented Nov 28, 2017

Landed in a177708, congrats on your first PR to Node.js!
❤️ 💚 💙 💛 💜

@maclover7 maclover7 closed this Nov 28, 2017

MylesBorins added a commit that referenced this pull request Dec 12, 2017

test: replace function with ES6 arrow function
PR-URL: #17306
Reviewed-By: Daijiro Wachi <daijiro.wachi@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Jon Moss <me@jonathanmoss.me>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Yuta Hiroto <hello@about-hiroppy.com>

MylesBorins added a commit that referenced this pull request Dec 12, 2017

test: replace function with ES6 arrow function
PR-URL: #17306
Reviewed-By: Daijiro Wachi <daijiro.wachi@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Jon Moss <me@jonathanmoss.me>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Yuta Hiroto <hello@about-hiroppy.com>

MylesBorins added a commit that referenced this pull request Dec 12, 2017

test: replace function with ES6 arrow function
PR-URL: #17306
Reviewed-By: Daijiro Wachi <daijiro.wachi@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Jon Moss <me@jonathanmoss.me>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Yuta Hiroto <hello@about-hiroppy.com>

@MylesBorins MylesBorins referenced this pull request Dec 12, 2017

Merged

v9.3.0 proposal #17631

gibfahn added a commit that referenced this pull request Dec 19, 2017

test: replace function with ES6 arrow function
PR-URL: #17306
Reviewed-By: Daijiro Wachi <daijiro.wachi@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Jon Moss <me@jonathanmoss.me>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Yuta Hiroto <hello@about-hiroppy.com>

gibfahn added a commit that referenced this pull request Dec 19, 2017

test: replace function with ES6 arrow function
PR-URL: #17306
Reviewed-By: Daijiro Wachi <daijiro.wachi@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Jon Moss <me@jonathanmoss.me>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Yuta Hiroto <hello@about-hiroppy.com>

@gibfahn gibfahn referenced this pull request Dec 20, 2017

Closed

v8.9.4 proposal #17772

gibfahn added a commit that referenced this pull request Dec 20, 2017

test: replace function with ES6 arrow function
PR-URL: #17306
Reviewed-By: Daijiro Wachi <daijiro.wachi@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Jon Moss <me@jonathanmoss.me>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Yuta Hiroto <hello@about-hiroppy.com>

@gibfahn gibfahn referenced this pull request Dec 20, 2017

Merged

v8.9.4 proposal #17774

@MylesBorins MylesBorins referenced this pull request Dec 20, 2017

Merged

v6.12.3 proposal #17776

msoechting added a commit to hpicgs/node that referenced this pull request Feb 5, 2018

test: replace function with ES6 arrow function
PR-URL: nodejs#17306
Reviewed-By: Daijiro Wachi <daijiro.wachi@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Jon Moss <me@jonathanmoss.me>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Yuta Hiroto <hello@about-hiroppy.com>

msoechting added a commit to hpicgs/node that referenced this pull request Feb 7, 2018

test: replace function with ES6 arrow function
PR-URL: nodejs#17306
Reviewed-By: Daijiro Wachi <daijiro.wachi@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Jon Moss <me@jonathanmoss.me>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Yuta Hiroto <hello@about-hiroppy.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment