FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

test: fix test stream finished by MoLow · Pull Request #43633 · nodejs/node · GitHub

/ node Public

test: fix test stream finished - #43633

Closed
MoLow wants to merge 1 commit into
nodejs:mainfrom
MoLow:fix-test-streap-finished
Closed

test: fix test stream finished#43633
MoLow wants to merge 1 commit into
nodejs:mainfrom
MoLow:fix-test-streap-finished

Conversation

MoLow commented Jun 30, 2022

Copy link
Copy Markdown
Member

fixes #43623 (Hopefully 🙌 )

nodejs-github-bot added needs-ci PRs that need a full CI run. test Issues and PRs related to the tests. labels Jun 30, 2022
tniessen added the request-ci Add this label to start a Jenkins CI on a PR. label Jun 30, 2022
github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Jun 30, 2022

Copy link
Copy Markdown
Collaborator

F3n67u commented Jun 30, 2022

Copy link
Copy Markdown
Contributor

Not fixed. test-stream-finished is still failing in linux https://ci.nodejs.org/job/node-test-commit-linux/46372/nodes=rhel8-x64/testReport/junit/(root)/test/parallel_test_stream_finished/

MoLow commented Jul 3, 2022

Copy link
Copy Markdown
Member Author

fixed in #43641

MoLow closed this Jul 3, 2022
MoLow deleted the fix-test-streap-finished branch July 3, 2022 07:27
This file contains hidden or 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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-ci PRs that need a full CI run. test Issues and PRs related to the tests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Investigate flaky test-stream-finished

4 participants


Back | FazBrowse Home | New Git URL