v7.20.4 #3613
Merged
v7.20.4 #3613
Conversation
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
This moves a handful of the smaller tests to using the new npm mock that uses the real actual npm object. It also extends the testing surface area of a few tests back down into the actual `process.spawn` that results, instead of anything internal to the code. Some dead code in `lib/test.js` was found during this, as well as an instance of a module throwing a string instead of an error object. PR-URL: #3463 Credit: @wraithgar Close: #3463 Reviewed-by: @nlf
* fix: properly handle top-level files when using strip * Avoid an unlikely but theoretically possible redos * WriteEntry backpressure * fix(unpack): always resume parsing after an entry error * fix(unpack): fix hang on large file on open() fail * fix: properly prefix hard links
* Clear progress bar which overlays confirm prompt
* fix: start retrying immediately, stop after 10 attempts
Contributor
|
test-only stuff seems like it shouldn't even really be in the changelog :-) |
Member
Author
Dead code in lib/test.js was removed, as well as a tweak to throw an error object instead of a string. That's in the longer description of the PR/commit. |
Contributor
|
ah, gotcha, that wasn't clear from the short description |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
v7.20.4 (2021-08-05)
BUG FIXES
6a8086e25#3463 fix(tests): move more tests to use real npm (@wraithgar)DEPENDENCIES
15fae4941[email protected]:745326de0[email protected]:e82bcd4e8[email protected]: