Skip to content

Conversation

@lpinca
Copy link
Member

@lpinca lpinca commented Jan 4, 2025

The original issue is likely the same as other tests that time out.

Refs: #54918
Refs: #53595
Refs: #53751

The original issue is likely the same as other tests that time out.

Refs: nodejs#54918
Refs: nodejs#53595
Refs: nodejs#53751
@nodejs-github-bot nodejs-github-bot added needs-ci PRs that need a full CI run. test Issues and PRs related to the tests. labels Jan 4, 2025
@codecov
Copy link

codecov bot commented Jan 4, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 88.54%. Comparing base (b3f82fe) to head (274cc5e).
Report is 28 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #56472      +/-   ##
==========================================
+ Coverage   88.53%   88.54%   +0.01%     
==========================================
  Files         657      657              
  Lines      190719   190761      +42     
  Branches    36605    36614       +9     
==========================================
+ Hits       168848   168910      +62     
+ Misses      15040    15037       -3     
+ Partials     6831     6814      -17     

see 59 files with indirect coverage changes

@aduh95 aduh95 added dont-land-on-v18.x dont-land-on-v20.x PRs that should not land on the v20.x-staging branch and should not be released in v20.x. dont-land-on-v22.x PRs that should not land on the v22.x-staging branch and should not be released in v22.x. labels Jan 4, 2025
@lpinca lpinca added the request-ci Add this label to start a Jenkins CI on a PR. label Jan 5, 2025
@github-actions github-actions bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Jan 5, 2025
@nodejs-github-bot

This comment was marked as outdated.

@nodejs-github-bot

This comment was marked as outdated.

@nodejs-github-bot
Copy link
Collaborator

nodejs-github-bot commented Jan 6, 2025

Copy link
Member

@mhdawson mhdawson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

jasnell pushed a commit that referenced this pull request Jan 6, 2025
The original issue is likely the same as other tests that time out.

Refs: #54918
Refs: #53595
Refs: #53751
PR-URL: #56472
Reviewed-By: Michael Dawson <[email protected]>
Reviewed-By: James M Snell <[email protected]>
@jasnell
Copy link
Member

jasnell commented Jan 6, 2025

Landed in 72537f5

@jasnell jasnell closed this Jan 6, 2025
@lpinca lpinca deleted the remove/flaky-test-pipe-file-to-http branch January 6, 2025 16:38
Ceres6 pushed a commit to Ceres6/node that referenced this pull request Jan 13, 2025
The original issue is likely the same as other tests that time out.

Refs: nodejs#54918
Refs: nodejs#53595
Refs: nodejs#53751
PR-URL: nodejs#56472
Reviewed-By: Michael Dawson <[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

dont-land-on-v20.x PRs that should not land on the v20.x-staging branch and should not be released in v20.x. dont-land-on-v22.x PRs that should not land on the v22.x-staging branch and should not be released in v22.x. 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.

5 participants