-
Notifications
You must be signed in to change notification settings - Fork 525
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Security Update broke test/fetch/pull-dont-push.js? #2697
Comments
That test was added by the security release and it's passing on main. Where is it failing? |
@mcollina |
The tests seems to be passing on main. Or maybe they are not run at all? On Mac the test pass on all version. |
This one PR I created is just adding an .npmrc file and the CI/CD pipeline is broken?! Maybe one of the commits merged after the security release broke it? I will investigate in few hours. |
Here is the fix: #2699 |
The unit test test/fetch/pull-dont-push.js is broken. I first thought it has something to do within my PR #2696 were I saw this first. But this issue is also on the default branch. I deleted locally my undici repo, cloned again and again the test runner hangs on this specific test.
I guess it has something to do with the security patches.
The text was updated successfully, but these errors were encountered: