Skip to content

Conversation

MoLow
Copy link
Member

@MoLow MoLow commented Jul 6, 2023

backport of #48057, #48034 and #48428

@nodejs-github-bot
Copy link
Collaborator

Review requested:

  • @nodejs/test_runner

@nodejs-github-bot nodejs-github-bot added lib / src Issues and PRs related to general changes in the lib or src directory. needs-ci PRs that need a full CI run. v18.x Issues that can be reproduced on v18.x or PRs targeting the v18.x-staging branch. labels Jul 6, 2023
@MoLow MoLow changed the title lib: support FORCE_COLOR for non TTY streams [v18.x backport] test_runner: align stdout and std error with and without --test Jul 6, 2023
@MoLow MoLow force-pushed the backport-align-stdout branch 2 times, most recently from 2042910 to 1ac9c1d Compare July 9, 2023 19:20
MoLow added 4 commits July 9, 2023 22:24
@MoLow MoLow force-pushed the backport-align-stdout branch from 1ac9c1d to c8cd582 Compare July 9, 2023 19:26
@MoLow MoLow added the request-ci Add this label to start a Jenkins CI on a PR. label Jul 9, 2023
@MoLow MoLow requested a review from danielleadams July 9, 2023 19:28
@github-actions github-actions bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Jul 9, 2023
@nodejs-github-bot
Copy link
Collaborator

@nodejs-github-bot
Copy link
Collaborator

@nodejs-github-bot
Copy link
Collaborator

@nodejs-github-bot
Copy link
Collaborator

@nodejs-github-bot
Copy link
Collaborator

@nodejs-github-bot
Copy link
Collaborator

@danielleadams
Copy link
Contributor

@MoLow does d0a39ca have a PR associated?

@MoLow
Copy link
Member Author

MoLow commented Jul 12, 2023

@MoLow does d0a39ca have a PR associated?

@danielleadams , yes. It is https://github.com/nodejs/node/pull/48057/commits which contains two commits

danielleadams pushed a commit that referenced this pull request Jul 12, 2023
PR-URL: #48034
Backport-PR-URL: #48684
Reviewed-By: Yagiz Nizipli <[email protected]>
Reviewed-By: Benjamin Gruenbaum <[email protected]>
danielleadams pushed a commit that referenced this pull request Jul 12, 2023
PR-URL: #48057
Backport-PR-URL: #48684
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Antoine du Hamel <[email protected]>
danielleadams pushed a commit that referenced this pull request Jul 12, 2023
PR-URL: #48057
Backport-PR-URL: #48684
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Antoine du Hamel <[email protected]>
danielleadams pushed a commit that referenced this pull request Jul 12, 2023
PR-URL: #48428
Backport-PR-URL: #48684
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Benjamin Gruenbaum <[email protected]>
@danielleadams
Copy link
Contributor

Landed in a2bfe02...5d910ca

@MoLow MoLow deleted the backport-align-stdout branch July 12, 2023 12:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lib / src Issues and PRs related to general changes in the lib or src directory. needs-ci PRs that need a full CI run. v18.x Issues that can be reproduced on v18.x or PRs targeting the v18.x-staging branch.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants