beacon-biosignals / SlackThreads.jl

Update a running Slack thread with text and attachments.
MIT License
2 stars 1 forks source link

Use Downloads.jl instead of shelling out to curl #5

Closed ericphanson closed 1 year ago

ericphanson commented 2 years ago

Downloads.jl is a stdlib in Julia 1.6+ that uses libcurl to make requests. We could use that instead of shelling out to curl, which would improve portability (user doesn’t need curl binary) and be slightly nicer since it can reuse connections and let other tasks run while it waits (although maybe run also let’s other tasks go while it waits, and maybe the curl binary can reuse connections, not sure).

ericphanson commented 1 year ago

I think we can close this by https://github.com/beacon-biosignals/SlackThreads.jl/pull/21