search

kevva / bin-wrapper

Binary wrapper that makes your programs seamlessly available as local dependencies
MIT License
152 stars 65 forks source link

Update most dependencies #60

Closed realityking closed 6 years ago

realityking commented 6 years ago

The exception being download, I'll open a separate PR for it once this one is merged. The unit tests are fighting me.

As an aside, now that the dependencies are all Promise-based, what do you think about changing the API of bin-wrapper to be Promise-based?

coreyfarrell commented 6 years ago

I think the update of download will require merge of kevva/download#163.

DesWurstes commented 6 years ago

Updating kevva/download solves tunnel-agent < 0.6.0 vulnerability, from bin-wrapper > download > caw > tunnel-agent

XhmikosR commented 6 years ago

Ping @kevva

rejas commented 6 years ago

If you need help @kevva testing and progressing these PRs and issues feel free to contact me, would be glad to help out (also with kevva/download#163)

realityking commented 6 years ago

@sindresorhus Could you have a look at this one?

sindresorhus commented 6 years ago

what do you think about changing the API of bin-wrapper to be Promise-based?

👍

sindresorhus commented 6 years ago

@rejas I'm sure @kevva could use some help. Triage and/or fix issues: https://github.com/kevva/bin-wrapper/issues?q=is%3Aissue+is%3Aopen+sort%3Aupdated-desc for example.

realityking commented 6 years ago

Thanks @sindresorhus! I'll follow-up with PRs for a Promise based API and the download dependency later this week :)

sindresorhus commented 6 years ago

I think the update of download will require merge of kevva/download#163.

New download version is published.

sindresorhus commented 6 years ago

@realityking Sounds good! 👌