-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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
fix: prefer fs/promises over promisify #7399
Conversation
This is likely to collide w/ my "remove tables" PR, I'll push it up tomorrow and would like to land that first. |
d8624ed
to
67b9431
Compare
67b9431
to
c796af3
Compare
Arborist had more than needed individual comments but it seems we aren't consistently using Also should it be |
The sync methods are not available from |
After reviewing again I was wrong, I remembered the |
ae83cb3
to
c796af3
Compare
No description provided.