Skip to content
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

Add detecting Web Stream to default (core) entry point #649

Merged
merged 14 commits into from
Aug 2, 2024

Conversation

Borewit
Copy link
Collaborator

@Borewit Borewit commented Jul 29, 2024

In addition to the fileTypeStream() converting a Node Readable stream, into a detecting stream, add support for web streams.

In Node.js you can either use a Node Readable stream, or a Web Stream.
Using the default entry point (core), just supports Web Stream.

@Borewit Borewit added the enhancement Add new functionality label Jul 29, 2024
@Borewit Borewit self-assigned this Jul 29, 2024
@Borewit Borewit force-pushed the extend-web-stream-with-file-type branch 3 times, most recently from 821e8be to df5b0a0 Compare July 30, 2024 21:02
@Borewit Borewit marked this pull request as ready for review July 30, 2024 21:03
@Borewit
Copy link
Collaborator Author

Borewit commented Jul 30, 2024

Please merge #650 & #651 first.

@Borewit Borewit force-pushed the extend-web-stream-with-file-type branch from df5b0a0 to d7fb48e Compare July 31, 2024 09:08
@Borewit
Copy link
Collaborator Author

Borewit commented Jul 31, 2024

This PR undo PR #651, as the constructor is not overridden, there is no need to re-declare it.

index.d.ts Outdated Show resolved Hide resolved
index.js Show resolved Hide resolved
test.js Outdated Show resolved Hide resolved
core.js Outdated Show resolved Hide resolved
core.js Outdated Show resolved Hide resolved
core.js Outdated Show resolved Hide resolved
@sindresorhus
Copy link
Owner

I have committed some changes, so please don't force push and override them.

@sindresorhus
Copy link
Owner

Ensure the readme and index.d.ts are in sync.

@Borewit Borewit force-pushed the extend-web-stream-with-file-type branch from cb9b5a5 to bff09c8 Compare August 1, 2024 08:37
@Borewit Borewit force-pushed the extend-web-stream-with-file-type branch from 9825a0b to 257bc99 Compare August 1, 2024 19:47
core.js Outdated Show resolved Hide resolved
core.js Outdated Show resolved Hide resolved
core.js Outdated Show resolved Hide resolved
@Borewit Borewit force-pushed the extend-web-stream-with-file-type branch from 53ec046 to f337a4c Compare August 2, 2024 08:55
index.d.ts Outdated Show resolved Hide resolved
index.d.ts Outdated Show resolved Hide resolved
@sindresorhus sindresorhus merged commit 2000141 into main Aug 2, 2024
6 checks passed
@sindresorhus sindresorhus deleted the extend-web-stream-with-file-type branch August 2, 2024 14:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Add new functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants