Skip to content
This repository has been archived by the owner on Aug 4, 2021. It is now read-only.

Use async fs methods #230

Merged
merged 1 commit into from Jun 26, 2019
Merged

Use async fs methods #230

merged 1 commit into from Jun 26, 2019

Conversation

kaksmet
Copy link
Contributor

@kaksmet kaksmet commented Jun 25, 2019

Fixes #147

Copy link
Member

@lukastaegert lukastaegert left a comment

Choose a reason for hiding this comment

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

Thanks, looks good!

src/index.js Outdated Show resolved Hide resolved
src/index.js Outdated Show resolved Hide resolved
@kaksmet
Copy link
Contributor Author

kaksmet commented Jun 26, 2019

Review comments have been addressed.

Copy link
Member

@lukastaegert lukastaegert left a comment

Choose a reason for hiding this comment

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

❤️

@lukastaegert lukastaegert merged commit 84471e4 into rollup:master Jun 26, 2019
@kaksmet kaksmet deleted the async branch June 26, 2019 07:33
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Use async fs instead of fs.*Sync calls
2 participants