pythonicrubyist / creek

Ruby library for parsing large Excel files.
http://rubygems.org/gems/creek
MIT License
388 stars 109 forks source link

Remove http dependency? #80

Closed fanfilmu closed 4 years ago

fanfilmu commented 5 years ago

Hello!

I would like to propose removing the http gem dependency entirely. From what I see, it's only used to download remote XLSX files, which, in my opinion, should be handled outside the gem by the devs (by their favourite http gem with 10 monkeypatches for their own super-secret security policy :) ).

Benefits that I see:

Personally, it would allow me to update Creek without regenerating VCRs and solving problems that come with it... 😇

Let me know what you think!

mdavidn commented 5 years ago

Yes, please. The http gem brings six transitive dependencies, including one native extension.