Skip to content
This repository has been archived by the owner on Mar 7, 2023. It is now read-only.

Use native Promises #69

Merged
merged 2 commits into from
Sep 27, 2018
Merged

Use native Promises #69

merged 2 commits into from
Sep 27, 2018

Conversation

timfish
Copy link
Contributor

@timfish timfish commented Sep 26, 2018

Node has had native promises since v4 so I don't think bluebird is required.

@MadLittleMods
Copy link
Owner

@timfish Seems fine. We should update the engines field in package.json, https://docs.npmjs.com/files/package.json#engines

@timfish
Copy link
Contributor Author

timfish commented Sep 27, 2018

Updated engines to >=4

@MadLittleMods
Copy link
Owner

Thanks @timfish ❤️

I looked through the code to make sure we weren't using any specific bluebird methods and ran the test suite ✔️

@MadLittleMods MadLittleMods merged commit 166302b into MadLittleMods:master Sep 27, 2018
@MadLittleMods
Copy link
Owner

MadLittleMods commented Sep 27, 2018

This is now part of v4.0.0 🚀 , https://github.com/MadLittleMods/node-usb-detection/blob/master/CHANGELOG.md#v400---2018-9-27

Waiting on the prebuild builds for the tag before pushing to npm, https://github.com/MadLittleMods/node-usb-detection/releases/tag/v4.0.0

Update: v4.0.0 published to npm, https://www.npmjs.com/package/usb-detection

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants