Skip to content

Support progressive reporting#141

Merged
sindresorhus merged 9 commits intosindresorhus:mainfrom
jopemachine:add-progress-event-emitter
May 16, 2022
Merged

Support progressive reporting#141
sindresorhus merged 9 commits intosindresorhus:mainfrom
jopemachine:add-progress-event-emitter

Conversation

@jopemachine
Copy link
Copy Markdown
Contributor

Fixes #133.

Implemented progressive reporting by referring https://github.com/sindresorhus/cpy#progress-reporting.

@jopemachine
Copy link
Copy Markdown
Contributor Author

The test seems failed on Windows due to this issue

@sindresorhus
Copy link
Copy Markdown
Owner

I no longer think attaching properties to a promise is the right approach. I think it's better to do a onProgress option.

Comment thread test.js Outdated
Comment thread index.d.ts
Comment thread test.js Outdated
Comment thread test.js Outdated
Comment thread readme.md Outdated
Comment thread readme.md Outdated
Comment thread readme.md Outdated
Comment thread index.d.ts Outdated
Comment thread index.d.ts Outdated
Comment thread index.d.ts Outdated
Comment thread index.d.ts Outdated
Comment thread index.d.ts
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.

Add progress event

2 participants