Skip to content

Update minipass#15

Merged
isaacs merged 3 commits intoisaacs:mainfrom
wraithgar:gar/deps-updates
Mar 25, 2026
Merged

Update minipass#15
isaacs merged 3 commits intoisaacs:mainfrom
wraithgar:gar/deps-updates

Conversation

@wraithgar
Copy link
Copy Markdown
Contributor

@wraithgar wraithgar commented Feb 19, 2026

This is a breaking change since the minipass it extends had breaking changes.

Engines was also updated to match current minipass

BREAKING CHANGE: engines.node is now ">=16 || 14 >=14.17"
Comment thread test/basic.js
f.on('data', d => {
sawData = true
t.equal(d, 'foo')
t.equal(d.toString('utf8'), 'foo')
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Casting here instead of passing encoding to get coverage for the "opts is just the callback" code path.

@isaacs isaacs force-pushed the gar/deps-updates branch from 6eedc09 to d850609 Compare March 25, 2026 16:53
@isaacs isaacs closed this in d850609 Mar 25, 2026
@isaacs isaacs merged commit d850609 into isaacs:main Mar 25, 2026
11 of 16 checks passed
@isaacs
Copy link
Copy Markdown
Owner

isaacs commented Mar 25, 2026

Ack, oops, published as patch, thinking that the port to TS was the breaking change. Is that going to cause a problem? I'm pretty sure npm is the only user, but I can roll it back.

@wraithgar wraithgar deleted the gar/deps-updates branch April 20, 2026 15:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants