Skip to content

[v20.x backport] test_runner: catch reporter errors - #49941

Closed
MoLow wants to merge 1 commit into
nodejs:v20.x-stagingfrom
MoLow:backport-print-report-errors
Closed

[v20.x backport] test_runner: catch reporter errors#49941
MoLow wants to merge 1 commit into
nodejs:v20.x-stagingfrom
MoLow:backport-print-report-errors

Conversation

@MoLow

Copy link
Copy Markdown
Member

Backporting #49646

PR-URL: nodejs#49646Fixes: nodejs#48937
Reviewed-By: Chemi Atlow <chemi@atlow.co.il>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Review requested:

  • @nodejs/test_runner

@nodejs-github-botnodejs-github-bot added needs-ci PRs that need a full CI run. test_runner Issues and PRs related to the test runner subsystem. v20.x labels Sep 28, 2023
@MoLow

MoLow commented Nov 8, 2023

Copy link
Copy Markdown
MemberAuthor

@nodejs/releasers can this land?

@RafaelGSS

Copy link
Copy Markdown
Member

@nodejs/releasers can this land?

We will land it when creating the release proposal and technically this PR needs Jenkins CI.

@MoLowMoLow added the request-ci Add this label to start a Jenkins CI on a PR. label Nov 12, 2023
@github-actionsgithub-actionsBot removed the request-ci Add this label to start a Jenkins CI on a PR. label Nov 12, 2023
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@targos

Copy link
Copy Markdown
Member

Commit landed cleanly after other ones and is part of #50682

@targostargos closed this Nov 12, 2023
@MoLow
MoLow deleted the backport-print-report-errors branch May 24, 2024 09:01
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-ciPRs that need a full CI run.test_runnerIssues and PRs related to the test runner subsystem.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@MoLow@nodejs-github-bot@RafaelGSS@targos@rluvaton