Skip to content

Conversation

@thefourtheye
Copy link
Contributor

This patch

  • issues a TAP plugin parsable message for non-darwin environments
  • uses const wherever applicable
  • moves the test to parallel

CI Run: https://jenkins-iojs.nodesource.com/job/node-test-pull-request/202/

@thefourtheye thefourtheye added the test Issues and PRs related to the tests. label Aug 28, 2015
@Fishrock123
Copy link
Contributor

cc @nodejs/build again

@mscdex mscdex added the fs Issues and PRs related to the fs subsystem / file system. label Sep 14, 2015
This patch

  - issues a TAP plugin parsable message on non darwin/windows boxes
  - uses `const` wherever applicable
  - moves the test to parallel
@thefourtheye thefourtheye force-pushed the improve-fs-watch-recursive branch from 579160a to fe8d32d Compare November 4, 2015 20:50
@thefourtheye
Copy link
Contributor Author

Rebased and doing CI: https://ci.nodejs.org/job/node-test-pull-request/671/

@thefourtheye
Copy link
Contributor Author

Wow. Almost all green. Any reviewers? cc @nodejs/collaborators ?

@thefourtheye
Copy link
Contributor Author

Bump!

@bnoordhuis
Copy link
Member

LGTM

1 similar comment
@jbergstroem
Copy link
Member

LGTM

@jasnell
Copy link
Member

jasnell commented Nov 10, 2015

LGTM!

thefourtheye added a commit that referenced this pull request Nov 10, 2015
This patch

  - issues a TAP plugin parsable message on non darwin/windows boxes
  - uses `const` wherever applicable
  - moves the test to parallel

PR-URL: #2599
Reviewed-By: Ben Noordhuis <[email protected]>
Reviewed-By: Johan Bergström <[email protected]>
Reviewed-By: James M Snell <[email protected]>
@jasnell
Copy link
Member

jasnell commented Nov 10, 2015

Landed in a28b938

@jasnell jasnell closed this Nov 10, 2015
@thefourtheye thefourtheye deleted the improve-fs-watch-recursive branch November 10, 2015 16:30
thefourtheye added a commit that referenced this pull request Nov 11, 2015
This patch

  - issues a TAP plugin parsable message on non darwin/windows boxes
  - uses `const` wherever applicable
  - moves the test to parallel

PR-URL: #2599
Reviewed-By: Ben Noordhuis <[email protected]>
Reviewed-By: Johan Bergström <[email protected]>
Reviewed-By: James M Snell <[email protected]>
@Fishrock123 Fishrock123 mentioned this pull request Nov 11, 2015
thefourtheye added a commit that referenced this pull request Nov 16, 2015
This patch

  - issues a TAP plugin parsable message on non darwin/windows boxes
  - uses `const` wherever applicable
  - moves the test to parallel

PR-URL: #2599
Reviewed-By: Ben Noordhuis <[email protected]>
Reviewed-By: Johan Bergström <[email protected]>
Reviewed-By: James M Snell <[email protected]>
@MylesBorins
Copy link
Contributor

landed in lts as 75b4613

thefourtheye added a commit that referenced this pull request Dec 4, 2015
This patch

  - issues a TAP plugin parsable message on non darwin/windows boxes
  - uses `const` wherever applicable
  - moves the test to parallel

PR-URL: #2599
Reviewed-By: Ben Noordhuis <[email protected]>
Reviewed-By: Johan Bergström <[email protected]>
Reviewed-By: James M Snell <[email protected]>
@jasnell jasnell mentioned this pull request Dec 17, 2015
thefourtheye added a commit that referenced this pull request Dec 17, 2015
This patch

  - issues a TAP plugin parsable message on non darwin/windows boxes
  - uses `const` wherever applicable
  - moves the test to parallel

PR-URL: #2599
Reviewed-By: Ben Noordhuis <[email protected]>
Reviewed-By: Johan Bergström <[email protected]>
Reviewed-By: James M Snell <[email protected]>
thefourtheye added a commit that referenced this pull request Dec 23, 2015
This patch

  - issues a TAP plugin parsable message on non darwin/windows boxes
  - uses `const` wherever applicable
  - moves the test to parallel

PR-URL: #2599
Reviewed-By: Ben Noordhuis <[email protected]>
Reviewed-By: Johan Bergström <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fs Issues and PRs related to the fs subsystem / file system. test Issues and PRs related to the tests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants