Skip to content

test: add mustCallAtLeast to test-fs-read-stream-inherit - #27457

Closed
johvin wants to merge 1 commit into
nodejs:masterfrom
johvin:test_patch
Closed

test: add mustCallAtLeast to test-fs-read-stream-inherit#27457
johvin wants to merge 1 commit into
nodejs:masterfrom
johvin:test_patch

Conversation

@johvin

Copy link
Copy Markdown
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

@nodejs-github-botnodejs-github-bot added the test Issues and PRs related to the tests. label Apr 28, 2019
@starkwangstarkwang added the code-and-learn Issues related to the Code-and-Learn events and PRs submitted during the events. label Apr 28, 2019
@ZYSzysZYSzys added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Apr 28, 2019
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@oyyd

oyyd commented Apr 29, 2019

Copy link
Copy Markdown
Contributor

please leave 👍 to fast-track

@oyydoyyd added the fast-track PRs that do not need to wait for 48 hours to land. label Apr 29, 2019
@oyyd

oyyd commented Apr 29, 2019

Copy link
Copy Markdown
Contributor

Landed in f5e2ad7, thank!

You can visit https://www.nodetodo.org/next-steps/ for more ideas to contribute.

@oyydoyyd closed this Apr 29, 2019
oyyd pushed a commit that referenced this pull request Apr 29, 2019
PR-URL: #27457
Reviewed-By: Weijia Wang <starkwang@126.com>
Reviewed-By: Yorkie Liu <yorkiefixer@gmail.com>
Reviewed-By: Yongsheng Zhang <zyszys98@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
targos pushed a commit that referenced this pull request Apr 29, 2019
PR-URL: #27457
Reviewed-By: Weijia Wang <starkwang@126.com>
Reviewed-By: Yorkie Liu <yorkiefixer@gmail.com>
Reviewed-By: Yongsheng Zhang <zyszys98@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
@targostargos mentioned this pull request May 6, 2019
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author readyPRs that have at least one approval, no pending requests for changes, and a CI started.code-and-learnIssues related to the Code-and-Learn events and PRs submitted during the events.fast-trackPRs that do not need to wait for 48 hours to land.testIssues and PRs related to the tests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants

@johvin@nodejs-github-bot@oyyd@Trott@yorkie@cjihrig@starkwang@ZYSzys